Skip to content

Upgrade GitHub Actions - #2106

Open
cclauss wants to merge 1 commit into
pypa:mainfrom
cclauss:patch-1
Open

Upgrade GitHub Actions#2106
cclauss wants to merge 1 commit into
pypa:mainfrom
cclauss:patch-1

Conversation

@cclauss

@cclauss cclauss commented Aug 14, 2026

Copy link
Copy Markdown

Do not encourage visitors to rely on out-of-date GitHub Actions.


📚 Documentation preview 📚: https://python-packaging-user-guide--2106.org.readthedocs.build/en/2106/

@cclauss
cclauss requested a review from webknjaz as a code owner August 14, 2026 21:38
@webknjaz
webknjaz enabled auto-merge August 14, 2026 23:07

steps:
- uses: actions/checkout@v6
- uses: actions/checkout@v7
persist-credentials: false
- name: Set up Python
uses: actions/setup-python@v6
uses: actions/setup-python@v7
run: python3 -m build
- name: Store the distribution packages
uses: actions/upload-artifact@v5
uses: actions/upload-artifact@v7
steps:
- name: Download all the dists
uses: actions/download-artifact@v6
uses: actions/download-artifact@v8
Do not encourage visitors to rely on out-of-date GitHub Actions.
auto-merge was automatically disabled August 14, 2026 23:56

Head branch was pushed to by a user without write access

@cclauss cclauss left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

% GH_TOKEN=$(gh auth token) zizmor --fix=all source/guides/github-actions-ci-cd-sample/publish-to-pypi.yml

The review comments were fixed, but should we go full-zizmor or leave the other violations as is?

Unfortunately, zizmor cannot autofix:
uses: pypa/gh-action-pypi-publish@release/v1
but perhaps these docs want to continue to advocate that.

uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
with:
name: python-package-distributions
path: dist/

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

zizmor: warning[excessive-permissions]: overly broad permissions

- name: Publish distribution 📦 to TestPyPI
uses: pypa/gh-action-pypi-publish@release/v1
with:
repository-url: https://test.pypi.org/legacy/

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

zizmor: warning[excessive-permissions]: overly broad permissions

name: python-package-distributions
path: dist/
- name: Publish distribution 📦 to PyPI
uses: pypa/gh-action-pypi-publish@release/v1

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Suggested change
uses: pypa/gh-action-pypi-publish@dc37677b2e1c63e2034f94d8a5b11f265b73ba33 # v1.14.2

name: python-package-distributions
path: dist/
- name: Publish distribution 📦 to TestPyPI
uses: pypa/gh-action-pypi-publish@release/v1

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Suggested change
uses: pypa/gh-action-pypi-publish@dc37677b2e1c63e2034f94d8a5b11f265b73ba33 # v1.14.2

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.

3 participants