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

Test sort options: `--sort-percentage` and `--sort-uncovered`. #269

merged 2 commits into from Aug 18, 2018


Copy link

mkurdej commented Jul 5, 2018

No description provided.

@mkurdej mkurdej force-pushed the mkurdej:test-sort branch from 7be57e4 to 1953cc5 Jul 5, 2018

This comment has been minimized.

Copy link

codecov bot commented Jul 5, 2018

Codecov Report

Merging #269 into master will increase coverage by 0.7%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #269     +/-   ##
+ Coverage   90.47%   91.18%   +0.7%     
  Files          13       13             
  Lines        1565     1565             
  Branches      269      269             
+ Hits         1416     1427     +11     
+ Misses         98       88     -10     
+ Partials       51       50      -1
Impacted Files Coverage Δ
gcovr/ 76.59% <0%> (+5.85%) ⬆️

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 b4b8fd9...1953cc5. Read the comment docs.

@mkurdej mkurdej closed this Jul 5, 2018
@mkurdej mkurdej reopened this Jul 5, 2018

This comment has been minimized.

Copy link
Contributor Author

mkurdej commented Aug 14, 2018


latk approved these changes Aug 18, 2018
@latk latk merged commit 1953cc5 into gcovr:master Aug 18, 2018
4 checks passed
4 checks passed
codecov/patch Coverage not affected when comparing b4b8fd9...1953cc5
codecov/project 91.18% (+0.7%) compared to b4b8fd9
continuous-integration/appveyor/pr AppVeyor build succeeded
continuous-integration/travis-ci/pr The Travis CI build passed
latk added a commit that referenced this pull request Aug 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
2 participants
You can’t perform that action at this time.