-
Notifications
You must be signed in to change notification settings - Fork 1
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
[PR] Codecov fix #117
[PR] Codecov fix #117
Conversation
@nelsonic please merge |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #117 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 6 6
Lines 62 62
=========================================
Hits 62 62 ☔ View full report in Codecov by Sentry. |
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.
Thanks for proactively reading the docs and creating this PR. ❤️
Super lame that they've made a breaking change. 🤦♂️
So ... we'll need to apply this fix to all our repos ... ⏳ 🙄
fixes the Codecov failing.
According to https://github.com/codecov/codecov-action#breaking-changes, with the new
v4
, tokenless uploads will cease to work.