Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Merged
gravyboat merged 1 commit into
streamlink:masterfrom
beardypig:docs-80-chars
Jun 15, 2018
Merged

help: reformat all the help text so that it is <80 chars#1754
gravyboat merged 1 commit into
streamlink:masterfrom
beardypig:docs-80-chars

Conversation

@beardypig

Copy link
Copy Markdown
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

codecov Bot commented Jun 6, 2018

Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
Member Author

@gravyboat I think this is OK to merge.

@gravyboat

Copy link
Copy Markdown
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.

2 participants