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

Update: codecov yaml #1393

Merged
merged 1 commit into from
Feb 8, 2022
Merged

Update: codecov yaml #1393

merged 1 commit into from
Feb 8, 2022

Conversation

eddiebergman
Copy link
Contributor

We previously had 87% code coverage so this is what I set codecov to use as the failing line. That PR also update coverage to measure branches taken of conditionals, which then lowered test coverage by 3%. This PR addresses that and sets the failing line to 84%.

@codecov
Copy link

codecov bot commented Feb 8, 2022

Codecov Report

Merging #1393 (74c895d) into development (7e7beb4) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development    #1393      +/-   ##
===============================================
- Coverage        84.48%   84.41%   -0.08%     
===============================================
  Files              146      146              
  Lines            11267    11267              
  Branches          1925     1925              
===============================================
- Hits              9519     9511       -8     
- Misses            1234     1242       +8     
  Partials           514      514              

Impacted file tree graph

@eddiebergman eddiebergman merged commit cfc0fac into development Feb 8, 2022
@mfeurer mfeurer deleted the update_codecov_yaml branch February 9, 2022 11:48
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 this pull request may close these issues.

1 participant