-
-
Notifications
You must be signed in to change notification settings - Fork 859
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
remove: need for hard coded token #9136
remove: need for hard coded token #9136
Conversation
✅ Deploy Preview for inventree-web-pui-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #9136 +/- ##
==========================================
+ Coverage 85.96% 86.13% +0.16%
==========================================
Files 1181 1180 -1
Lines 52209 52105 -104
Branches 2184 2184
==========================================
- Hits 44884 44882 -2
+ Misses 6767 6666 -101
+ Partials 558 557 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@SchrodingersGat after merging this we can remove the secret CODECOV_TOKEN |
codecov supports github OIDC for uplad so we can remove the hard-coded token