Skip to content

Conversation

@willnet
Copy link
Member

@willnet willnet commented Dec 6, 2025

I released a new gem myself and confirmed that bundle exec rake release works, so I added that to MAINTAINING.md.
I’m not sure what exactly should go in this TODO section, so for now I only wrote what I could.

@willnet willnet requested a review from a team December 6, 2025 13:27
Copy link
Contributor

@brendon brendon left a comment

Choose a reason for hiding this comment

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

Looks good. I didn't realise GitHub releases was a manual process. I assume there are ways to automate this along with the gem release task?

@brendon
Copy link
Contributor

brendon commented Dec 7, 2025

There is this as an option too: https://github.com/rubygems/release-gem

@joshbuker
Copy link
Member

@willnet I think the todo was for stuff like how to push with MFA enabled on rubygems, and maybe switching to a CI style release pattern like @brendon suggested with the release-gem GitHub action.

Ideally, I'd like to have CI confirm that everything is set correctly (Sorcery::VERSION, primarily), then auto push the gem when creating a new release via GitHub releases.

@brendon
Copy link
Contributor

brendon commented Dec 8, 2025

Hopefully we all have MFA enabled on the rubygems side? I use the fingerprint sensor which does require following a link from the command line but it's not that bad.

@joshbuker
Copy link
Member

joshbuker commented Dec 8, 2025

Yes, I'm pretty MFA is required for all Sorcery maintainers on RubyGems. I was referring more to the step of actually passing the MFA code while pushing the gem via CLI.

@brendon
Copy link
Contributor

brendon commented Dec 8, 2025

Yes, I'm pretty MFA is required for all Sorcery maintainers on RubyGems. I was referring more to the step of actually passing the MFA code while pushing the gem via CLI.

True, I think it's probably pretty self explanatory as the CLI makes it pretty clear what it wants. At least for the key url, from memory the OTC request is a bit more terse.

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.

3 participants