-
Notifications
You must be signed in to change notification settings - Fork 306
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 codecov bug #2637
Comments
Hm...it seems like uploading coverage from forks can only occur at a maximum rate. I suspect this may have been happening before, and we're only noticing it now because I set codecov to fail when uploading coverage fails. I'm not quite sure how to address this, other than to let the Codecov action succeed in CI even if it can't be uploaded (i.e., to revert the setting that I changed). If we do that, then we should make sure that the corresponding weekly test that does coverage should fail when it is not able to upload stuff. It may be possible to set up Codecov tokens that work for forks too. I might not be able to get to this this week, so in the meantime it may be best to think of this as an allowed failure. Thank you for raising this! |
We've also been getting this error, seemingly intermittently:
|
The following bug is consistently appearing in CI, most recently on #2636
@namurphy I know you were working on this - what's going on?
The text was updated successfully, but these errors were encountered: