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

coverage test upload appears to require a Codecov token #52

Closed
cjyetman opened this issue Apr 30, 2024 · 2 comments
Closed

coverage test upload appears to require a Codecov token #52

cjyetman opened this issue Apr 30, 2024 · 2 comments

Comments

@cjyetman
Copy link
Member

Not sure how/when/why it stopped working, but it appears that the uploading of test coverage results to Codecov is not happening (even though the workflow passes without error), which then means the badge on the README is out of date.

message in the workflow log says
https://github.com/RMI-PACTA/pacta.scenario.data.preparation/actions/runs/8892294430/job/24416055743#step:5:53

[1] "Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected available in 3597 seconds."

Documentation for {covr} suggests one can/should use a token through one of a few methods, either explicitly through the token argument, or
https://covr.r-lib.org/reference/codecov.html

a codecov upload token, if NULL then following external sources will be checked in this order:

the environment variable ‘CODECOV_TOKEN’. If it is empty, then

package will look at directory of the package for a file codecov.yml. File must have codecov > section where field token is set to a token that will be used.

The existing codecov.yml here does not have a token specified.

@jdhoffa do you have any idea what's going on here?

@jdhoffa
Copy link
Member

jdhoffa commented Apr 30, 2024

I'm not sure, I don't recall making a lot of use of the codecov.yml action in the past (although I see the utility).

I think the CODECOV token would be a good candidate for something to set at the RMI-PACTA org level, I can look into creating one

@cjyetman
Copy link
Member Author

cjyetman commented May 6, 2024

resolved by #53

@cjyetman cjyetman closed this as completed May 6, 2024
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

No branches or pull requests

2 participants