-
Notifications
You must be signed in to change notification settings - Fork 867
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
chore: Add Codecov token parameter #10459
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10459 +/- ##
==========================================
+ Coverage 74.31% 79.40% +5.08%
==========================================
Files 536 547 +11
Lines 23189 23623 +434
Branches 4056 4061 +5
==========================================
+ Hits 17234 18758 +1524
+ Misses 4853 3695 -1158
- Partials 1102 1170 +68 ☔ View full report in Codecov by Sentry. |
bbbe2eb
to
f796f72
Compare
Note: So it need to set |
5a7c297
to
4f6f756
Compare
I've added What's changed from default Codecov setting
TODO
If these checks are not displayed. |
4f6f756
to
bd8e9ea
Compare
Change It's required because forked repository's main branch CI is failed when So don't use |
After PR(#10456) is merged to
main
.Following error occurred on CI job on main branch.
For protected branch.
Tokenless Uploads for Public Repositories
seems not works.So I've added parameter that is used before.