Skip to content

Fix and improve help for --cov-report#132

Merged
ionelmc merged 1 commit intopytest-dev:masterfrom
blueyed:fix-cov-report-help
Jan 3, 2017
Merged

Fix and improve help for --cov-report#132
ionelmc merged 1 commit intopytest-dev:masterfrom
blueyed:fix-cov-report-help

Conversation

@blueyed
Copy link
Contributor

@blueyed blueyed commented Sep 8, 2016

--cov-report= to not output any error is hard to guess.

`--cov-report=` to not output any error is hard to guess.
@ionelmc
Copy link
Member

ionelmc commented Sep 8, 2016

Neat argparse trick! But maybe we should make a more explicit --cov-report=none option? What do you think?

@blueyed
Copy link
Contributor Author

blueyed commented Sep 8, 2016

Neat argparse trick!

It is mentioned in the README, too.

--cov-report=none sounds good.
I can image most people are using it only to generate the .coverage file, e.g. on Travis, and then use other services on top. But making none the default is probably a bit too much?!

@ionelmc
Copy link
Member

ionelmc commented Sep 8, 2016

Well I was just asking for opinion. I'm going to merge this meanwhile.

@ionelmc ionelmc merged commit 790e8ce into pytest-dev:master Jan 3, 2017
@blueyed blueyed deleted the fix-cov-report-help branch January 4, 2017 00:49
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