Skip to content

Conversation

@KyleKing
Copy link
Collaborator

@KyleKing KyleKing commented Oct 19, 2025

Attempt to address CI failure: https://github.com/executablebooks/mdformat-footnote/actions/runs/18628585373/job/53110081204 and add Dependabot to keep Actions up to date in the future

Current runner version: '2.328.0'
Runner Image Provisioner
Operating System
Runner Image
GITHUB_TOKEN Permissions
Secret source: Actions
Prepare workflow directory
Prepare all required actions
Getting action download info
Download action repository 'actions/download-artifact@v4' (SHA:d3f86a106a0bac45b974a628896c90dbdf5c8093)
Download action repository 'sigstore/gh-action-sigstore-python@v2.1.1' (SHA:61f6a500bbfdd9a2a339cf033e5421951fbc1cd2)
Getting action download info
Error: This request has been automatically failed because it uses a deprecated version of `actions/upload-artifact: v3`. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

@KyleKing KyleKing requested a review from gaige October 19, 2025 14:20
@KyleKing KyleKing self-assigned this Oct 19, 2025
path: dist/
- name: Sign the dists with Sigstore
uses: sigstore/gh-action-sigstore-python@v2.1.1
uses: sigstore/gh-action-sigstore-python@v3.0.0
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Using this action isn't recommended (https://github.com/sigstore/gh-action-sigstore-python/blob/be60bbea7f8770c9c9673a8fabafec00df7f458f/README.md) because the SigStore version is encapsulated by the pypa/gh-action-pypi-publish action

The files modified during the pypi-publish action should be uploaded directly, right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Although, I might just not have a good understanding of what the best practices are now and haven't been following the OIDC changes closely

Copy link
Collaborator

Choose a reason for hiding this comment

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

They're separate issues. The first action pushes to pypi and signs there. The second one signs the artifacts as part of the github release.

@gaige gaige merged commit 330b8ba into master Oct 19, 2025
13 checks passed
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