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] Update and fix coverage test #51

Merged
merged 3 commits into from
Feb 13, 2025
Merged

[ci] Update and fix coverage test #51

merged 3 commits into from
Feb 13, 2025

Conversation

diraol
Copy link
Collaborator

@diraol diraol commented Feb 12, 2025

Update the test coverage workflow to use the codecov v5, the Token is not required anymore (for public repos if the "github org" is configured not to request such token). Nevertheless we have configured the token for this repo as a secret.

hsvab
hsvab previously approved these changes Feb 12, 2025
Now, on the v5 of codecov, the Token is not required anymore (for public
repos if the "github org" is configured not to request such token).
@diraol
Copy link
Collaborator Author

diraol commented Feb 13, 2025

We will have to merge this, as is, with it failing on the R-CMD-check.yaml test because it's another issue that will be addressed in another PR.

@hsvab hsvab merged commit 099a059 into main Feb 13, 2025
9 of 10 checks passed
@diraol diraol deleted the dro/fix_coverage branch February 13, 2025 12:36
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