Open woodruffw opened 3 months ago
xref https://github.com/actions/starter-workflows/issues/2344, https://github.com/github/docs/pull/32146
CC @di @jhutchings1
CC @webknjaz as well, as the maintainer of gh-action-pypi-publish
🙂
@juliandunn @N-Usha FYI, I've been in discussion with @woodruffw, @di and the PyPI team about this. This change switches from using token based authentication to OIDC, and would be a great benefit to the security posture of this community. Let me know if you have any questions I can help with to get the review prioritized by Actions engineering.
@woodruffw love that this is moving somewhere! Thank you for getting to this sooner than me :) I've been frustrated with how many people get pre-historic workflows by default and don't even know it...
One extra thing to consider — it might be useful to also stick Sigstore signing right into the starter. OTOH, giving people a link to the guide might be an alternative.
I've been in discussion with @woodruffw, @di and the PyPI team about this.
Linking some context: https://github.com/pypa/gh-action-pypi-publish/pull/123#issuecomment-1982201999
One extra thing to consider — it might be useful to also stick Sigstore signing right into the starter. OTOH, giving people a link to the guide might be an alternative.
I'd personally like to shy away from suggesting the Sigstore action here for now, if only because (with PEP 740) it'll become obsolete and integrated directly into the publishing flow 🙂
This should be good to go. I think it might make sense to review and land this before https://github.com/github/docs/pull/32146, since the workflow changes here will need to be reflected there as well.
Gentle ping for review here!
(@jhutchings1 I'm calling in that promise 😉)
Another gentle ping for review here: I believe this should be merged before https://github.com/github/docs/pull/32146 goes forwards, since those docs will link to this workflow.
@nebuk89 Can you help get this prioritized?
Fixes #2344.
Tasks
For all workflows, the workflow:
.yml
file with the language or platform as its filename, in lower, kebab-cased format (for example,docker-image.yml
). Special characters should be removed or replaced with words as appropriate (for example, "dotnet" instead of ".NET").GITHUB_TOKEN
so that the workflow runs successfully.For CI workflows, the workflow:
ci
directory.ci/properties/*.properties.json
file (for example,ci/properties/docker-publish.properties.json
).push
tobranches: [ $default-branch ]
andpull_request
tobranches: [ $default-branch ]
.release
withtypes: [ created ]
.docker-publish.yml
).Some general notes:
actions
organization, or