Skip to content

Improve help text#350

Merged
asottile merged 3 commits intopre-commit:masterfrom
brettmcintosh:help_text_fixes
Mar 9, 2016
Merged

Improve help text#350
asottile merged 3 commits intopre-commit:masterfrom
brettmcintosh:help_text_fixes

Conversation

@brettmcintosh
Copy link
Copy Markdown

@brettmcintosh brettmcintosh commented Mar 8, 2016

Add choices to --color help text.
Fix typo in --allow-unstaged-config.

Improve help text
23a140a 
Add choices to --color help text.
Fix typo in --allow-unstaged-config.
@asottile
Copy link
Copy Markdown
Member

asottile commented Mar 8, 2016

you've failed the pre-commit hooks, please fix and :shipit: :)

asottile added a commit that referenced this pull request Mar 9, 2016
@asottile
Merge pull request #350 from brettmcintosh/help_text_fixes
77a1a29 
Improve help text
@asottile asottile merged commit 77a1a29 into pre-commit:master Mar 9, 2016
@asottile
Copy link
Copy Markdown
Member

asottile commented Mar 9, 2016

Thanks!

@asottile
Copy link
Copy Markdown
Member

asottile commented Mar 9, 2016

fwiw, I took it one step further with argparse's metavar: eb6da4a

Now looks like this:

$ pre-commit run --help
...
  --color {auto,always,never}
                        Whether to use color in output. Defaults to `auto`.
...

@brettmcintosh
Copy link
Copy Markdown
Author

brettmcintosh commented Mar 9, 2016

Much better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@brettmcintosh @asottile