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

Add sget to the release artifacts #745

Merged
merged 1 commit into from
Sep 23, 2021

Conversation

rgreinho
Copy link
Contributor

During the release process, builds sget along cosign, signs, verifies
and publishes the binaries.

Fixes #725

@rgreinho rgreinho force-pushed the issues725/sget-release branch from 38ca57f to f515323 Compare September 21, 2021 19:42
@dekkagaijin dekkagaijin requested a review from cpanato September 21, 2021 19:49
@cpanato
Copy link
Member

cpanato commented Sep 22, 2021

cool, thanks @rgreinho!

@dlorenc @dekkagaijin I will update the release pipeline to include this in the release as well ok?

Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

small fix, thanks fro this PR

During the release process, builds `sget` along cosign, signs, verifies
and publishes the binaries.

Fixes sigstore#725

Signed-off-by: Rémy Greinhofer <[email protected]>
@rgreinho rgreinho force-pushed the issues725/sget-release branch from 9e0bcc5 to d253ab8 Compare September 22, 2021 13:47
@dlorenc
Copy link
Member

dlorenc commented Sep 22, 2021

@cpanato it looks like your comment was addressed. Merge if you're happy!

@cpanato
Copy link
Member

cpanato commented Sep 23, 2021

I'm always happy :)

Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

thanks!!

@cpanato cpanato merged commit 4ed933a into sigstore:main Sep 23, 2021
@github-actions github-actions bot added this to the v1.3.0 milestone Sep 23, 2021
rgreinho added a commit to rgreinho/cosign that referenced this pull request Sep 23, 2021
After pushing sigstore#745, the signing/verifying mechamnism for
the created binaries was broken. This patch fixes the issue and removes
the useless duplication, making the CI workflow DRYer.
@rgreinho rgreinho mentioned this pull request Sep 23, 2021
rgreinho added a commit to rgreinho/cosign that referenced this pull request Sep 23, 2021
After pushing sigstore#745, the signing/verifying mechanism for
the created binaries was broken. This patch fixes the issue and removes
the useless duplication, making the CI workflow DRYer.
rgreinho added a commit to rgreinho/cosign that referenced this pull request Sep 23, 2021
After pushing sigstore#745, the signing/verifying mechanism for
the created binaries was broken. This patch fixes the issue and removes
the useless duplication, making the CI workflow DRYer.

Signed-off-by: Rémy Greinhofer <[email protected]>
dlorenc pushed a commit that referenced this pull request Sep 23, 2021
After pushing #745, the signing/verifying mechanism for
the created binaries was broken. This patch fixes the issue and removes
the useless duplication, making the CI workflow DRYer.

Signed-off-by: Rémy Greinhofer <[email protected]>
@rgreinho rgreinho deleted the issues725/sget-release branch November 9, 2021 16:31
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.

release sget
3 participants