add option --cov-summary to show coverage summary in terminal#140
add option --cov-summary to show coverage summary in terminal#140flniu wants to merge 1 commit intopytest-dev:masterfrom
Conversation
|
Wouldn't an addition to |
|
At first, I thought about
So I choose an easier way which add an option but has the least impact on existing logic. |
|
My two concerns are:
|
|
Well, I agree not to add lots of switches. I thought about our requirement again. In current version, when
What we need is: when required coverage is reached, also output a message like:
How about this plan? |
|
If you mean changing |
|
Yes, that is what I mean. I'll work on it. This pr can be closed then. |
In our case, we want to see a brief summary about the total coverage and required coverage in CI console, instead of printing a full term report or opening the html report. So I add an option
--cov-summaryto output coverage summary in terminal.