Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.8k
[DO NOT MERGE before adding the token] chore: updatecodecov-action tov4#1605
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
codecov-commenter commentedFeb 2, 2024
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@## master #1605 +/- ##======================================= Coverage 82.44% 82.44% ======================================= Files 377 377 Lines 19747 19747 Branches 2884 2885 +1 ======================================= Hits 16281 16281 Misses 3466 3466 ☔ View full report in Codecov by Sentry. |
appgurueu commentedFeb 2, 2024
It seems that in order to add the token, I would have to be logged in (on the codecov site) to accesshttps://app.codecov.io/gh/TheAlgorithms/JavaScript/settings ? I'm slightly uncomfortable with this as it requests permission to "act on my behalf": |
vil02 commentedFeb 2, 2024
Yes.
I agree. |
appgurueu commentedMay 23, 2024
I bit the bullet and authorized CodeCov to obtain the token, but it turns out that I can't edit repo secrets since I do not have "admin" permissions. This will require@Panquesito7 or@AnupKumarPanwar I believe. |
raklaptudirm commentedJun 23, 2024
@appgurueu You should have admin access to the repository now |
vil02 commentedJun 23, 2024
Just in case: please make sure that |
appgurueu commentedJun 23, 2024
Why would dependabot need the codecov token? It seems to work as-is:https://app.codecov.io/github/TheAlgorithms/JavaScript/commit/9010481c2930d65621f18f41c216f29fceaa8807 |
vil02 commentedJun 23, 2024
Without it, the |

Describe your change:
Same asTheAlgorithms/Rust#670 - it contains the description of the problem.
Before merging, please add the token fromhttps://app.codecov.io/gh/TheAlgorithms/JavaScript/settings as
CODECOV_TOKENinto the repo secrets.Checklist: