Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Include coverage percentage in Cobertura reports #3034

Merged
merged 4 commits into from
Apr 19, 2023
Merged

Conversation

Porges
Copy link
Member

@Porges Porges commented Apr 19, 2023

Closes #2824. ReportGenerator (apparently) does not use this information, but I noticed that it was missing when using PyCobertura to generate a report.

@Porges Porges marked this pull request as ready for review April 19, 2023 00:49
@Porges Porges force-pushed the cobertura-rates branch 2 times, most recently from 6d56749 to 35929d1 Compare April 19, 2023 02:04
@Porges Porges enabled auto-merge (squash) April 19, 2023 05:13
@Porges Porges requested review from chkeita and tevoinea April 19, 2023 20:44
@codecov-commenter
Copy link

codecov-commenter commented Apr 19, 2023

Codecov Report

Merging #3034 (b566882) into main (aa28550) will increase coverage by 0.63%.
The diff coverage is 97.67%.

@@            Coverage Diff             @@
##             main    #3034      +/-   ##
==========================================
+ Coverage   30.79%   31.42%   +0.63%     
==========================================
  Files         306      306              
  Lines       36599    36633      +34     
==========================================
+ Hits        11270    11513     +243     
+ Misses      25329    25120     -209     
Impacted Files Coverage Δ
src/agent/coverage/src/source.rs 12.35% <71.42%> (+2.78%) ⬆️
src/agent/coverage/src/cobertura.rs 100.00% <100.00%> (+100.00%) ⬆️

... and 2 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Porges Porges merged commit 0432829 into main Apr 19, 2023
@Porges Porges deleted the cobertura-rates branch April 19, 2023 22:28
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request May 4, 2023
@nharper285 nharper285 mentioned this pull request May 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Calculate line rates for Cobertura coverage
3 participants