-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Set minimum required Ruby version to 2.7 #24992
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -69,7 +69,7 @@ public class RubyClientCodegen extends AbstractRubyCodegen { | |
| protected String specFolder = "spec"; | ||
| protected String libFolder = "lib"; | ||
| @Setter protected String gemLicense = "unlicense"; | ||
| @Setter protected String gemRequiredRubyVersion = ">= 1.9"; | ||
| @Setter protected String gemRequiredRubyVersion = ">= 2.7"; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P3: This field default does not reach the generated gemspec, so changing it (and the old Prompt for AI agents |
||
| @Setter protected String gemHomepage = "https://openapitools.org"; | ||
| @Setter protected String gemSummary = "A Ruby SDK for the REST API"; | ||
| @Setter protected String gemDescription = "This gem maps to a REST API"; | ||
|
|
@@ -164,7 +164,7 @@ public RubyClientCodegen() { | |
| defaultValue("unlicense")); | ||
|
|
||
| cliOptions.add(new CliOption(GEM_REQUIRED_RUBY_VERSION, "gem required Ruby version. "). | ||
| defaultValue(">= 2.4")); | ||
| defaultValue(">= 2.7")); | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P3: Raising the minimum Ruby version to 2.7 leaves the ruby-client RuboCop template stale: Prompt for AI agents |
||
|
|
||
| cliOptions.add(new CliOption(GEM_HOMEPAGE, "gem homepage. "). | ||
| defaultValue("https://openapi-generator.tech")); | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: The new
>= 2.7default contradicts theCliOptiondefault for this same option, which still showsdefaultValue(">= 2.4")on line 167. That CLI default is what users see inconfig-helpand what is auto-generated intodocs/generators/ruby.md(row:|gemRequiredRubyVersion|gem required Ruby version. | |>= 2.4|), so the documented default remains below the new minimum and users generating from that default (e.g. via a copied JSON config) still get gems requiring>= 2.4. Update theCliOptiondefault to>= 2.7and regenerate the ruby generator docs so the advertised default matches the new minimum.Prompt for AI agents