Skip to content
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

help: reformat all the help text so that it is <80 chars #1754

Merged
merged 1 commit into from Jun 15, 2018

Conversation

beardypig
Copy link
Member

On smaller consoles I noticed that the help text gets a bit garbled with text wrapping, so I fixed it up a bit :)

And I noticed a mistake here: https://github.com/streamlink/streamlink/compare/master...beardypig:docs-80-chars?expand=1#diff-2b258c59b2cac0b01e6240cd4024be63R244

There are probably other mistakes, I might spend some time reading the argument help again :)

@beardypig beardypig added enhancement WIP Work in process labels Jun 6, 2018
@codecov
Copy link

codecov bot commented Jun 6, 2018

Codecov Report

Merging #1754 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1754   +/-   ##
=======================================
  Coverage   50.02%   50.02%           
=======================================
  Files         238      238           
  Lines       13990    13990           
=======================================
  Hits         6998     6998           
  Misses       6992     6992

@gravyboat
Copy link
Member

Took a quick look over everything and it looks good.

@beardypig beardypig removed the WIP Work in process label Jun 7, 2018
@beardypig
Copy link
Member Author

@gravyboat I think this is OK to merge.

@gravyboat
Copy link
Member

@beardypig Yep thanks for the reminder and the contribution.

@gravyboat gravyboat merged commit 60ff843 into streamlink:master Jun 15, 2018
mkbloke pushed a commit to mkbloke/streamlink that referenced this pull request Aug 18, 2020
help: reformat all the help text so that it is <80 chars
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants