Skip to content

Use PyPI Trusted Publishing for releases - #5311

Closed
saitcakmak wants to merge 1 commit into
meta:mainfrom
saitcakmak:export-D128356370
Closed

saitcakmak wants to merge 1 commit into
meta:mainfrom
saitcakmak:export-D128356370

Conversation

@saitcakmak

Copy link
Copy Markdown
Contributor

Summary:

What Changed

  • Grant the deploy job id-token: write for PyPI OIDC and explicitly retain contents: read for checkout.
  • Remove the user and password publishing inputs, including the PYPI_TOKEN secret reference.
  • Preserve the pinned publishing action, release triggers, test dependency, and TestPyPI workflow.

The production PyPI trusted publisher has already been registered by the author. This updates the internal source of OSS .github/workflows/deploy.yml.

Differential Revision: D128356370

Summary:
## What Changed
- Grant the `deploy` job `id-token: write` for PyPI OIDC and explicitly retain `contents: read` for checkout.
- Remove the `user` and `password` publishing inputs, including the `PYPI_TOKEN` secret reference.
- Preserve the pinned publishing action, release triggers, test dependency, and TestPyPI workflow.

The production PyPI trusted publisher has already been registered by the author. This updates the internal source of OSS `.github/workflows/deploy.yml`.

Differential Revision: D128356370
@meta-cla meta-cla Bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Oct 29, 2026
@meta-codesync

meta-codesync Bot commented Oct 29, 2026

Copy link
Copy Markdown

@saitcakmak has exported this pull request. If you are a Meta employee, you can view the originating Diff in D128356370.

@meta-codesync

meta-codesync Bot commented Oct 29, 2026

Copy link
Copy Markdown

This pull request has been merged in 5a7b336.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Do not delete this pull request or issue due to inactivity. Merged meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant