Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prettier readme #1006
Prettier readme #1006
Changes from 11 commits
2b5b6e3
e7ca3ef
9e3c9e1
d1015c2
6b70933
90d7ef0
589e6e6
7af97fd
2a4f304
b6f7801
f14d811
84746cb
3c4ece8
63ea1cb
30ac69d
bb0a4f0
909c94a
89c7af5
7da3994
6320dc0
bd63031
8372434
3a1ca0b
e12a2fa
1ad5cfb
b4b50d4
6013abd
f020df3
f9ee3ba
2a55ee0
a1d2050
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For simplicity for the end user, some of whom may not be as technologically savvy, the original readme's links should be used
Directing users to the ffmpeg.org website adds space for error
See highlighted CORRECT link for windows, but most users would probably go for the big green "Download" button at the top of the page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also change
ffmpeg
toFFmpeg
as per FFmpeg's style guide.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure if that helps a whole lot. If the user is unable to navigate the FFmpeg site, then I doubt they would be able to navigate the Windows site. Plus, since those are unofficial builds, I imagine they are subject to change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm understood. Check with @Mikhail-Zex for the final call ig :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But do change
ffmpeg
toFFmpeg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Silverarmor Thanks for noticing, will do.
@MikhailZex what do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think both are exceedingly important. Do what you think is correct.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To download a playlist, run:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling Error.
Seperating --> Separating
Possibly change phrasing to