Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a delta-coverage summary report (txt and/or md) #534

Closed
manfred-brands opened this issue Jul 15, 2022 · 1 comment · Fixed by #535
Closed

Add a delta-coverage summary report (txt and/or md) #534

manfred-brands opened this issue Jul 15, 2022 · 1 comment · Fixed by #535

Comments

@manfred-brands
Copy link
Contributor

For a pull-request the coverage should not go down. Either more test on existing code is added or new code is accompanied with corresponding tests.

Given a 'main' branch coverage in a history.xml format (from a report generator run on the main branch)
Then run report generator on the PR branch.

Create a delta report sort of the one from coverlet but then matching the wording of your other summary reports.
Instead of one, it would have three columns: main, branch, delta
Roughly like:

Generated on: 15/07/2022 - 15:26:

Item                        Main                    Branch                  Delta
Coverage date:              15/07/2022 - 15:21      15/07/2022 - 15:25
Parser                      Cobertura               Cobertura
Assemblies                  1                       1                       0
Classes:                    1                       1                       0
Files:                      1                       1                       0
Line coverage:              14.2%                   57.1%                   42.9%
Covered lines               1                       4                       3
Uncovered lines:            6                       3                       -3
Coverable lines:            7                       7                       0
Total lines:                19                      19                      0

Simple.Calculator           14.2%                   57.1%                   42.9%
  Simple.Calculator         14.2%                   57.1%                   42.9%
@danielpalme
Copy link
Owner

I merged your PR and also added a MarkdownDeltaSummary report.
New release will be available within the next days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants