Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: remove release attestations #514

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

samypr100
Copy link
Contributor

See https://github.com/astral-sh/python-build-standalone/actions/runs/13165689836/job/36745165697

This remove release attestations for now until problem with glob resulting in no matches is identified

@zanieb zanieb added the ci:skip label Feb 5, 2025
@zanieb zanieb merged commit 99c99ab into astral-sh:main Feb 5, 2025
46 checks passed
@samypr100 samypr100 deleted the revert-attestations-on-releases branch February 5, 2025 23:19
zanieb pushed a commit that referenced this pull request Feb 11, 2025
## Summary

Reverts #514

This adds back support for release attestations after asset uploads.

Turns out extended glob is not supported via subject-path given how the
action searches the directories and applies the glob, so instead we
declare multiple simple globs since subject-path supports that too.

## Test Plan

A full release was tested with about 155 artifacts (had to limit release
triplets since free runners have limited resources) which resulted in 76
subjects created due to filtering.

See release here:
https://github.com/samypr100/python-build-standalone/releases/tag/20250205

See run here:
https://github.com/samypr100/python-build-standalone/actions/runs/13220724442/job/36905418053

See attestations here:
https://github.com/samypr100/python-build-standalone/attestations/4870998
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants