-
Notifications
You must be signed in to change notification settings - Fork 224
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
ignore coverage for benchmarks and examples #462
Conversation
codecov should be posting coverage diffs on the PRs by the way. @dwrensha not sure if you need to toggle anything on over at https://app.codecov.io/github/capnproto/capnproto-rust? Occasionally the codecov bot is impacted by rate limiting of the github API it uses so it could just be related to that |
i'll leave this as a draft until i can confirm the report is updating as expected |
I clicked around a bunch, bunch I don't think I changed any settings. I did notice on #461 that codecov failed on the upload step like this:
Weirdly, the overall run was still reported as a success. |
Ah yeah, that's the gitlab API rate limiting issue. If you add the repo's GitHub token to the action that apparently resolves that. There's some docs on the issue and the solution here- https://community.codecov.com/t/upload-issues-unable-to-locate-build-via-github-actions-api/3954/1 I'll give it a go on this PR when I get a moment |
I logged into codecov.com and gave the app permission on this repo. Maybe that will help. |
i've also added the token to the github action, but you'll need to add the token to the repo 'secrets' for that to work EDIT: seems to be working even without the token, whatever you did seems to have worked. Might still be worth adding the token though, in order to avoid that rate limiting issue. What do you think? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #462 +/- ##
==========================================
+ Coverage 48.54% 51.20% +2.65%
==========================================
Files 80 69 -11
Lines 34529 33623 -906
==========================================
+ Hits 16762 17216 +454
+ Misses 17767 16407 -1360 ☔ View full report in Codecov by Sentry. |
ignore benchmarks and examples which are artificially lowering the coverage.