-
Notifications
You must be signed in to change notification settings - Fork 868
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
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 |
@yufeih |
bd8e9ea
to
7773bb2
Compare
This comment was marked as outdated.
This comment was marked as outdated.
376f6b3
to
f80cef2
Compare
f80cef2
to
c9f1962
Compare
@yufeih This PR will is expected to resolve CI issue on |
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.