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

🔧 Codecov Config Update #137

Merged
merged 1 commit into from
Oct 3, 2022
Merged

🔧 Codecov Config Update #137

merged 1 commit into from
Oct 3, 2022

Conversation

burgholzer
Copy link
Member

Per default, codecov reports the coverage status on a PR once the first report is uploaded. However, if there is more than one job tracking coverage (C++ and Python, for example), then this incorrectly reports a lowered coverage until both reports have been received.

Conveniently, codecov offers an option to delay the notification so that it only posts the notification after both reports have been received.

Signed-off-by: burgholzer <burgholzer@me.com>
@burgholzer burgholzer added usability Anything related to usability continuous integration Anything related to the CI setup labels Oct 3, 2022
@burgholzer burgholzer self-assigned this Oct 3, 2022
@burgholzer burgholzer enabled auto-merge (squash) October 3, 2022 08:50
@codecov
Copy link

codecov bot commented Oct 3, 2022

Codecov Report

Merging #137 (9707ed1) into main (00de22a) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #137   +/-   ##
=====================================
  Coverage   91.2%   91.2%           
=====================================
  Files         20      20           
  Lines       1731    1731           
  Branches     414     414           
=====================================
  Hits        1580    1580           
  Misses       151     151           

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

@burgholzer burgholzer merged commit 8afa37d into main Oct 3, 2022
@burgholzer burgholzer deleted the codecov-notification branch October 3, 2022 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
continuous integration Anything related to the CI setup usability Anything related to usability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant