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 sigstore-ruby, archive ruby-sigstore #441

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

haydentherapper
Copy link
Contributor

A new implementation of the Ruby client has been implemented and will be donated to the Sigstore org. The previous implementation is no longer maintained, so it will be archived.

Also adds maintainers for the new repo based on offline discussion.

Closes sigstore/TSC#50

Summary

Release Note

Documentation

A new implementation of the Ruby client has been implemented and will be
donated to the Sigstore org. The previous implementation is no longer
maintained, so it will be archived.

Also adds maintainers for the new repo based on offline discussion.

Closes sigstore/TSC#50

Signed-off-by: Hayden Blauzvern <hblauzvern@google.com>
@haydentherapper haydentherapper requested a review from a team as a code owner May 13, 2024 14:55
@haydentherapper
Copy link
Contributor Author

cc @woodruffw @segiddins

@haydentherapper
Copy link
Contributor Author

sync-changes failing at HEAD, filed #442

@woodruffw
Copy link
Member

Thanks @haydentherapper!

@haydentherapper
Copy link
Contributor Author

haydentherapper commented May 20, 2024

I realized the failed test here is because we're deleting a user, which is WAI. However because we updated pulumi recently, we have a diff as seen in https://github.com/sigstore/community/actions/runs/8905874264. Working with a TSC member to kick off a sync from HEAD before pushing this through, then we'll coordinate on the repo transfer.

@@ -2341,6 +2341,53 @@ repositories:
dismissalRestrictions:
- sigstore-rs-codeowners
webCommitSignoffRequired: true
- name: sigstore-ruby
Copy link
Member

Choose a reason for hiding this comment

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

this will be a new repo? i opened and that redirect to ruby-sigstore

let sync that, maybe need some manual operations

@bobcallaway
Copy link
Member

what else do we need to do here to get this closed out?

@haydentherapper
Copy link
Contributor Author

Sync output now looks good, only has the changes specified here. Remaining steps:

  1. @segiddins to initiate transfer to sigstore/sigstore-ruby
  2. Admin unlocks user being deleted
  3. Admin manually syncs pulumi
  4. Verify with @segiddins that repo settings are as expected

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.

Donation of Ruby implementation
4 participants