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

Bump codecov/codecov-action to v4 #378

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

antoninbas
Copy link
Member

We need to use a token to upload coverage data.
We can also disable coverage data search, as we already provide the list of files (a single file in our case) to include in the report.

Copy link

codecov bot commented Sep 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.49%. Comparing base (a5e552a) to head (dd72ad0).
Report is 8 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #378      +/-   ##
==========================================
+ Coverage   62.61%   69.49%   +6.88%     
==========================================
  Files          27       25       -2     
  Lines        3838     3819      -19     
==========================================
+ Hits         2403     2654     +251     
+ Misses       1246      973     -273     
- Partials      189      192       +3     
Flag Coverage Δ
integration-tests 53.02% <ø> (+2.00%) ⬆️
unit-tests 68.57% <ø> (+6.67%) ⬆️

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

see 27 files with indirect coverage changes

We need to use a token to upload coverage data.
We can also disable coverage data search, as we already provide the list
of files (a single file in our case) to include in the report.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
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.

2 participants