Skip to content

Opt-in for MFA requirement - #192

Open
tagliala wants to merge 1 commit into
AaronC81:masterfrom
tagliala:security/opt-in-for-mfa
Open

Opt-in for MFA requirement#192
tagliala wants to merge 1 commit into
AaronC81:masterfrom
tagliala:security/opt-in-for-mfa

Conversation

@tagliala

@tagliala tagliala commented Jul 2, 2026

Copy link
Copy Markdown

Make the gem more secure by requiring that all privileged operations by any of the owners require OTP.

Additionally, explicitly set minimum required Ruby version to 3.0, as commander ~> 5.0 requires Ruby >= 3.0

Ref:

Comment thread sord.gemspec
Comment on lines +16 to +18
spec.metadata = { 'rubygems_mfa_required' => 'true' }

spec.required_ruby_version = '>= 3.0'

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

there is not a clear preference of " over ', so I went for '

Make the gem more secure by requiring that all privileged operations by
any of the owners require OTP.

Additionally, explicitly set minimum required Ruby version to 3.0, as
commander ~> 5.0 requires Ruby >= 3.0

Ref:
- https://guides.rubygems.org/mfa-requirement-opt-in/
- castwide/solargraph#1215
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.

1 participant