Skip to content

Conversation

@KonradIT
Copy link
Contributor

The readme file has some markdown syntax, for instance the - character one line blow the headings, and the * to make a list. I added .md to readme, now is readme.md and it looks great. See: https://github.com/KonradIT/FFmpeg/blob/master/README.md

Konrad

@michaelni
Copy link
Member

From IRC:
llogan: might be kind of weird to have one be markdown style and the others not
llogan: INSTALL, etc

@KonradIT
Copy link
Contributor Author

Shall I add .md to the other files? Or close PR and issue?

@ghost
Copy link

ghost commented May 28, 2014

Hi. Thanks for the contribution. If you markup and rename the other files referenced in README (LICENSE and INSTALL), then I'll be happy to merge it.

@KonradIT
Copy link
Contributor Author

Ok, so I added the missing .md extentions and added more markdown syntax for titles, lists, headings... New commits are:

@KonradIT
Copy link
Contributor Author

Line 6 fixed by 2fb3ef0

@KonradIT
Copy link
Contributor Author

@ghost
Copy link

ghost commented May 28, 2014

What about using blah instead of blah? There should be a blank line above 2. and above 3. in INSTALL.md. That should be all of the requested changes.

@KonradIT
Copy link
Contributor Author

Here you go: b3a8e09

@michaelni
Copy link
Member

squashed in 3 patches and applied
thanks

@michaelni michaelni closed this May 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants