Skip to content
This repository has been archived by the owner on Jan 12, 2021. It is now read-only.

fix help #39

Merged
merged 4 commits into from
May 19, 2020
Merged

fix help #39

merged 4 commits into from
May 19, 2020

Conversation

orsinium
Copy link
Member

Close #36

@webknjaz
Copy link

This probably needs a test

@orsinium
Copy link
Member Author

Too hard to test, flake8 raises sys.exit after showing help. Contributions are welcome, as always. And contributions with tests are welcome twice :)

@webknjaz
Copy link

Well, maybe call in the subprocess and check if something is in the stdout... Also, sys.exit() raises SystemExit internally. So I guess it's

pytest.raises(SystemExit, match=...):
    ...

@orsinium orsinium merged commit ff12802 into master May 19, 2020
@orsinium orsinium deleted the fix-help branch May 19, 2020 09:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Mention verbosity CLI args in the help hints
2 participants