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 README.md with codecov badge #731

Merged
merged 1 commit into from
Jun 28, 2023
Merged

Update README.md with codecov badge #731

merged 1 commit into from
Jun 28, 2023

Conversation

iamemilio
Copy link
Contributor

Adds codecov badge to Readme

@codecov-commenter
Copy link

codecov-commenter commented Jun 12, 2023

Codecov Report

Merging #731 (2e8c23b) into develop (53cd82a) will decrease coverage by 10.58%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop     #731       +/-   ##
============================================
- Coverage    93.70%   83.13%   -10.58%     
============================================
  Files           41      133       +92     
  Lines         2003    11918     +9915     
============================================
+ Hits          1877     9908     +8031     
- Misses          97     1762     +1665     
- Partials        29      248      +219     

see 92 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mirackara
Copy link
Contributor

just noticing that this is reducing the codecov. Is the readme included in the codecov report?

No. If any tests fail, there won't be a code coverage reported to codecov for the failing tests. That's why we are seeing different code coverage percentages

@iamemilio iamemilio merged commit 730c443 into develop Jun 28, 2023
@iamemilio iamemilio deleted the codecov-badge branch August 5, 2024 17:24
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.

3 participants