-
Notifications
You must be signed in to change notification settings - Fork 252
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: Fix Codecov check #989
Conversation
Codecov ReportAttention:
... and 6 files with indirect coverage changes 📢 Thoughts on this report? Let us know!. |
`cargo tarpaulin` has changed the case of one of its config options.
959ec26
to
7b53b71
Compare
One of the Codecov jobs succeeded, the other failed. I think the failure is due to the 10 minute timeout on building perhaps? Gonna try bumping that. |
Hmm, this time both Codecov runs failed, so the timeout isn't the issue. The error I saw before is now shown in both of them, which I thought was
|
Based on google/oss-fuzz#10174 (comment), I believe the problem is more likely an OOM. Nothing we can really do about this without bumping the size of the GitHub Actions runner to one that costs money. So for now I'll remove the timeout commit and we can just merge the fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK, modulo that this just clears the way for codecov to hit the next bug.
cargo tarpaulin
has changed the case of one of its config options.