8000 Change file level percent values in JSON summary report by Spacetown · Pull Request #570 · gcovr/gcovr · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Change file level percent values in JSON summary report #570

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

Merged
merged 8 commits into from
Mar 19, 2022

Conversation

Spacetown
Copy link
Member

On file level the range was from 0.0 to 1.0 and on summary level from 0.0 to 100.0. Now it's always from 0.0 to 100.0.

Closes #517

On file level the range was from 0.0 to 1.0 and on summary level from 0.0 to 100.0. Now it's always from 0.0 to 100.0.
@codecov
Copy link
codecov bot commented Mar 16, 2022

Codecov Report

Merging #570 (007ef43) into master (fccbab4) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #570   +/-   ##
=======================================
  Coverage   95.70%   95.70%           
=======================================
  Files          23       23           
  Lines        3305     3309    +4     
  Branches      610      610           
=======================================
+ Hits         3163     3167    +4     
  Misses         69       69           
  Partials       73       73           
Flag Coverage Δ
ubuntu-18.04 94.49% <100.00%> (+<0.01%) ⬆️
windows-2019 95.32% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
gcovr/utils.py 91.15% <100.00%> (-0.08%) ⬇️
gcovr/writer/csv.py 100.00% <100.00%> (ø)
gcovr/writer/json.py 96.66% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fccbab4...007ef43. Read the comment docs.

Copy link
Member
@latk latk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, except that the CSV test now seems to have changed? Do you have any idea what is going on there??

@Spacetown Spacetown merged commit aa8828e into gcovr:master Mar 19, 2022
@Spacetown Spacetown deleted the fix_percent_values_in__json_summary branch March 19, 2022 19:30
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.

Percentage values in json-summary report
2 participants
0