-
Notifications
You must be signed in to change notification settings - Fork 30
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
Codecov Tokenless upload #1155
base: main
Are you sure you want to change the base?
Codecov Tokenless upload #1155
Conversation
f003574
to
635d671
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1155 +/- ##
=========================================
Coverage 80.76% 80.76%
Complexity 2948 2948
=========================================
Files 400 400
Lines 15104 15104
Branches 1021 1021
=========================================
Hits 12199 12199
Misses 2288 2288
Partials 617 617
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
b9145a2
to
0c3acd7
Compare
Signed-off-by: Andre Kurait <akurait@amazon.com>
f9d9180
to
986b024
Compare
Pending fix codecov/codecov-action#1704 |
Since this is waiting on a downstream fix, going to mark this as draft until we know its ready to merge. |
We moved to a more complex workflow_run trigger due to issues codecov had with tokenless uploads for the past few months.
This was reported in codecov/codecov-action#1580 and recently resolved with version
5.0.7
.This change moves back to tokenless uploads which reduces risk and makes it easier to test any future updates and adds some support for codecov updates even with some test failures.
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.