Skip to content
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

Use PyPI Trusted Publisher for releases #1500

Merged
merged 1 commit into from
May 16, 2023
Merged

Use PyPI Trusted Publisher for releases #1500

merged 1 commit into from
May 16, 2023

Conversation

joerick
Copy link
Contributor

@joerick joerick commented May 15, 2023

Copy link
Contributor

@henryiii henryiii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Czaki
Copy link
Contributor

Czaki commented May 15, 2023

Did you define the pypi environment in repository settings?

@henryiii
Copy link
Contributor

It will create it the first time it’s used. Though if you want to control who can release, you should create or edit it. :)

@henryiii henryiii merged commit 301dca2 into main May 16, 2023
23 of 25 checks passed
@henryiii henryiii deleted the joerick-patch-1 branch May 16, 2023 17:29
@webknjaz
Copy link
Member

control who can release

Or put a cooldown timer, for example...

@@ -27,6 +27,9 @@ jobs:
needs: [dist]
runs-on: ubuntu-latest
if: github.event_name == 'release' && github.event.action == 'published'
environment: pypi
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A nicer syntax could be

Suggested change
environment: pypi
environment:
name: pypi
url: https://pypi.org/p/cibuildwheel

which makes the URL show up in the workflow graph UI (and some other places). I like rendering the versioned URL even, when I have a version variable available...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants