-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add trusted publishing workflows for TestPyPI and PyPI deployments #840
Conversation
Codecov Report
@@ Coverage Diff @@
## main #840 +/- ##
==========================================
- Coverage 93.80% 91.73% -2.08%
==========================================
Files 59 59
Lines 6235 6246 +11
==========================================
- Hits 5849 5730 -119
- Misses 386 516 +130
|
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.
don’t understand how this doesn’t need secrets but very ok with that
This is looking good. Will merge when the actions clear and open a new PR to prepare the release. |
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Follows up on comments in #839
To-Do List
Type of change
How Has This Been Tested?
Checklist (while developing)