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

Release verification with both (.sig, .crt) + .sigstore #517

Merged
merged 2 commits into from
Mar 6, 2023
Merged

Conversation

jleightcap
Copy link
Contributor

@jleightcap jleightcap commented Mar 3, 2023

Closes #509

Signed-off-by: Jack Leightcap <jack.leightcap@trailofbits.com>
@@ -91,7 +100,7 @@ jobs:
needs: [build]
name: Generate build provenance
permissions:
actions: read # To read the workflow path.
actions: read # To read the workflow path.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

(Ran prettier over source)

@jleightcap
Copy link
Contributor Author

Not sure what the correct procedure is for kicking off a test release build -- verified the contents of release.yml manually on local machine. Let me know what further testing is needed.

@jleightcap jleightcap marked this pull request as ready for review March 3, 2023 23:34
@jleightcap jleightcap changed the title Release verification with both (.sig, .crt) + .sigstore` Release verification with both (.sig, .crt) + .sigstore Mar 3, 2023
Copy link
Member

@woodruffw woodruffw left a comment

Choose a reason for hiding this comment

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

LGTM!

@woodruffw
Copy link
Member

Not sure what the correct procedure is for kicking off a test release build

We can merge here and do a release candidate -- that would probably be the easiest way.

@woodruffw
Copy link
Member

I'll shepherd this + do a RC.

@woodruffw woodruffw enabled auto-merge (squash) March 6, 2023 15:05
@woodruffw woodruffw merged commit 7114e5f into main Mar 6, 2023
@woodruffw woodruffw deleted the jl/509 branch March 6, 2023 15:07
@jleightcap
Copy link
Contributor Author

Thanks Will!

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.

Update our release process to produce .sigstore bundle files
2 participants