Skip to content

Conversation

woodruffw
Copy link
Member

Like with Trusted Publishing we need at least one of these, but not necessarily both. We need to gracefully handle the scenario where at least one (both not both) is missing.

xref SWIFTSIM/swiftgalaxy#26

CC @di

Like with Trusted Publishing we need at least one of these,
but not necessarily both. We need to gracefully handle the
scenario where at least one (both not both) is missing.

xref SWIFTSIM/swiftgalaxy#26

Signed-off-by: William Woodruff <william@trailofbits.com>
@woodruffw woodruffw requested a review from DarkaMaul April 3, 2025 18:00
@woodruffw woodruffw self-assigned this Apr 3, 2025
Signed-off-by: William Woodruff <william@trailofbits.com>
Signed-off-by: William Woodruff <william@trailofbits.com>
Copy link
Collaborator

@facutuesca facutuesca left a comment

Choose a reason for hiding this comment

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

Thanks for fixing it, LGTM!

@woodruffw woodruffw merged commit 5736314 into main Apr 3, 2025
8 checks passed
@woodruffw woodruffw deleted the ww/allow-missing-ref branch April 3, 2025 19:14
woodruffw added a commit to trail-of-forks/warehouse that referenced this pull request Apr 4, 2025
See pypi/pypi-attestations#109.

See SWIFTSIM/swiftgalaxy#26.

Signed-off-by: William Woodruff <william@trailofbits.com>
di pushed a commit to pypi/warehouse that referenced this pull request Apr 4, 2025
See pypi/pypi-attestations#109.

See SWIFTSIM/swiftgalaxy#26.

Signed-off-by: William Woodruff <william@trailofbits.com>
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.

2 participants