Skip to content

Comments

Add build provenance attestation#57

Open
kevinbackhouse wants to merge 1 commit intoGitHubSecurityLab:mainfrom
kevinbackhouse:attestation
Open

Add build provenance attestation#57
kevinbackhouse wants to merge 1 commit intoGitHubSecurityLab:mainfrom
kevinbackhouse:attestation

Conversation

@kevinbackhouse
Copy link
Contributor

This is better because it lets users verify the artifact with https://cli.github.com/manual/gh_attestation_verify

Copilot AI review requested due to automatic review settings February 20, 2026 22:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request replaces Sigstore-based artifact signing with GitHub's native build provenance attestation, enabling users to verify published packages using the GitHub CLI (gh attestation verify). The change improves the security workflow by creating attestations before publishing artifacts, ensuring they're immediately available when packages are downloaded.

Changes:

  • Added attestations: write permission to both publishing workflows
  • Replaced sigstore/gh-action-sigstore-python step with actions/attest-build-provenance
  • Moved attestation/signing step to occur before PyPI publishing (previously occurred after)

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/publish-to-testpypi.yaml Added attestations permission and replaced sigstore signing with build provenance attestation before TestPyPI publishing
.github/workflows/publish-to-pypi.yaml Added attestations permission and replaced sigstore signing with build provenance attestation before PyPI publishing

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kevinbackhouse kevinbackhouse marked this pull request as ready for review February 20, 2026 22:53
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