Skip to content

docs: fix three GitHub Actions examples that fail when copied - #832

Open
simonvanlierde wants to merge 1 commit into
pyOpenSci:mainfrom
simonvanlierde:fix-actions-examples
Open

simonvanlierde wants to merge 1 commit into
pyOpenSci:mainfrom
simonvanlierde:fix-actions-examples

Conversation

@simonvanlierde

Copy link
Copy Markdown
Contributor

Hi!

Three workflow examples fail when copied: one isn't valid YAML, one pins a checkout commit that doesn't exist, and one runs a nox session the nox page never defines.

  • tutorials/trusted-publishing.md: in Step 3, uses:, with: and run: sat two spaces deeper than name:, so the YAML doesn't parse. I lined them up, and moved checkout and setup-python to v7 to match the example workflow further down the page.
  • tests/run-tests.md: the checkout SHA is 39 characters, and GitHub has no such commit. It now pins v7.0.1, the same SHA as examples/pure-hatch.
  • tests/tests-ci.md: nox -s tests-<version> is now test-<version>, the session run-tests-nox.md defines. I also bumped checkout, setup-python and codecov-action to v7; codecov's README says v7 needs token: ${{ secrets.CODECOV_TOKEN }}.

Tests: nox -s docs-test passes, and so do the translated builds. Every YAML block on the trusted-publishing page parses. I left Python 3.9 in both matrices (end of life since October 2025), since dropping it is your call.

Best,

Simon

- trusted-publishing: align step keys so the Step 3 YAML parses, and
  move its checkout and setup-python to v7, as the example workflow uses
- run-tests: pin actions/checkout to a commit that exists (v7.0.1)
- tests-ci: run the nox session the nox page defines (test, not tests),
  and update checkout, setup-python and codecov-action
@netlify

netlify Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for python-package-guide ready!

Name Link
🔨 Latest commit 4e6dcff
🔍 Latest deploy log https://app.netlify.com/projects/python-package-guide/deploys/6adbabc8adb3dc4b64dc3097
😎 Deploy Preview https://deploy-preview-832--python-package-guide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

This branch has not been deployed

No deployments
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.

1 participant