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

ci: downgrade 'codecov-action' version to v3 #184

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

rst0git
Copy link
Member

@rst0git rst0git commented Jul 19, 2024

codecov-action@v4 requires an organization-level upload token instead of a single repository upload token. Thus, uploading codecov report fails with the following error when triggered by the push event of a merged pull request:

Commit creating failed: {"detail":"You do not have permission to perform this action."}

We are temporarily downgrading from v4 to v3 until we can generate an organization-level upload token.

@rst0git rst0git force-pushed the codecov-fix branch 2 times, most recently from d488a6a to 51b1d84 Compare July 19, 2024 15:23
@adrianreber adrianreber reopened this Jul 19, 2024
`codecov-action@v4` requires an organization-level upload token instead
of a single repository upload token [1]. Thus, uploading codecov report
fails with the following error when triggered by the push event of a
merged pull request:

	Commit creating failed: {"detail":"You do not have permission to perform this action."}

We are temporarily downgrading from v4 to v3 until we can generate
an organization-level upload token.

[1] https://docs.codecov.com/docs/codecov-uploader#upload-token

Signed-off-by: Radostin Stoyanov <[email protected]>
Copy link

codecov bot commented Jul 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.13%. Comparing base (76a006a) to head (ce0a56d).
Report is 8 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #184      +/-   ##
==========================================
- Coverage   50.13%   48.13%   -2.00%     
==========================================
  Files          21       21              
  Lines        1889     2279     +390     
==========================================
+ Hits          947     1097     +150     
- Misses        810     1050     +240     
  Partials      132      132              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@adrianreber adrianreber merged commit b2a1cfe into checkpoint-restore:master Jul 19, 2024
14 of 15 checks passed
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.

2 participants