-
Notifications
You must be signed in to change notification settings - Fork 429
chore: Update to Ruby v3.3 and jekyll v4.3 #2364
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
base: master
Are you sure you want to change the base?
Changes from all commits
2635584
cd36197
f5e5195
c14c6c9
9b7f3a4
d82ee47
7594690
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 | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -12,6 +12,12 @@ Metrics/PerceivedComplexity: | |||||||
| Max: 12 | ||||||||
| Metrics/CyclomaticComplexity: | ||||||||
| Max: 12 | ||||||||
| # Jekyll plugins legitimately define multiple modules/classes in one file | ||||||||
| Style/OneClassPerFile: | ||||||||
| Enabled: false | ||||||||
| # load_from_cache returns bool as success/failure indicator but also has side effects | ||||||||
| Naming/PredicateMethod: | ||||||||
| Enabled: false | ||||||||
|
||||||||
| Enabled: false | |
| AllowedMethods: | |
| - load_from_cache |
Copilot
AI
Apr 28, 2026
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.
AllCops.TargetRubyVersion is still set to 3.1, but this PR updates the toolchain/lockfile to Ruby 3.3.x. Update TargetRubyVersion to match the new supported Ruby version so RuboCop enables the right parser and rules.
| TargetRubyVersion: 3.1 | |
| TargetRubyVersion: 3.3 |
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -63,3 +63,7 @@ gem 'webrick' | |||||||
|
|
||||||||
| # Required on Ruby 2.6 (polyfill) | ||||||||
| gem 'ruby-next' | ||||||||
|
|
||||||||
| # no longer part of the default gems in Ruby v3.4.0+ | ||||||||
|
||||||||
| # no longer part of the default gems in Ruby v3.4.0+ | |
| # Added for compatibility with Ruby 3.4+, where these are no longer default gems. | |
| # Keeping them here is harmless on earlier supported Ruby versions as well. |
Copilot
AI
Apr 28, 2026
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.
The Gemfile still declares ruby '>= 3.1.0', but this PR updates the Pixi environment and lockfile to Ruby 3.3.3. Consider bumping the Gemfile Ruby requirement to the same minimum (or otherwise reconciling the supported Ruby versions) to avoid confusing/unsupported Bundler setups.
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.
Style/OneClassPerFileis disabled globally, which will also hide issues outside of Jekyll plugins. Consider scoping this to the plugin code only (e.g., viaExclude: ['_plugins/**/*.rb']) so the cop still applies to the rest of the repository.