-
Notifications
You must be signed in to change notification settings - Fork 359
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: upgrade Codecov to v4 #941
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #941 +/- ##
==========================================
+ Coverage 63.52% 63.59% +0.06%
==========================================
Files 146 146
Lines 11937 11937
==========================================
+ Hits 7583 7591 +8
+ Misses 3885 3878 -7
+ Partials 469 468 -1 ☔ View full report in Codecov by Sentry. |
Ok that's already promising - I'm assuming it worked because PRs for open-source forks don't need an upload token; I'll wait for @another-rex or @oliverchang to confirm the |
Ok yup so maybe it will still be broken for direct pushes... |
@G-Rath Token has been added now |
This will hopefully resolve the issues we're having with the v3 action, but requires the `CODECOV_TOKEN` secret to be configured per [this document](https://docs.codecov.com/docs/adding-the-codecov-token) which needs to be done by someone with repository permissions to set secrets.
This will hopefully resolve the issues we're having with the v3 action, but requires the
CODECOV_TOKEN
secret to be configured per this document which needs to be done by someone with repository permissions to set secrets.