-
Notifications
You must be signed in to change notification settings - Fork 119
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
ci: use a trusted publisher token for publishing to PyPI #1061
ci: use a trusted publisher token for publishing to PyPI #1061
Conversation
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.
Looks good. Let's just tweak a couple of names. However, let's wait to merge this till you hear back from test.pypi.org support and have ownership of ops
.
Co-authored-by: Ben Hoyt <benhoyt@gmail.com>
Co-authored-by: Ben Hoyt <benhoyt@gmail.com>
|
It looks like the backlog for these requests goes back roughly one year 😞. Some get handled "out of queue" - I did find one that's exactly the same situation and it was one of the "out of queue" ones, so I've adjusted the issue summary to make it clearer that it's only test.pypi.org, but that doesn't seem like it always gets quicker action. So I think for at least this PR, we should just go ahead with it and can address any issues if they happen. |
Yep, sounds right to me, thanks! |
Remove the use of the PyPI secret in the publish workflow, in favour of using the short-lived OIDC token via the Trusted Publisher system.
Also add a (manually triggered) workflow to publish to test.pypi.org to be able to validate that publishing changes (like this one) work as expected, without needing to actually publish.
Fixes #1021