Skip to content

feat: Ruby 4 support - #116

Merged
khvn26 merged 3 commits into
mainfrom
ci/expand-ruby-matrix
Oct 29, 2026
Merged

khvn26 merged 3 commits into
mainfrom
ci/expand-ruby-matrix

Conversation

@khvn26

@khvn26 khvn26 commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Adds Ruby 3.4 and 4.0, the two most recent supported releases, to the CI matrix.

Ruby 4.0 needed a few dependency changes:

  • Bump the locked faraday-retry from 2.2.1 to 2.4.0: earlier versions declare required_ruby_version < 4, so bundle install failed. 2.4.0 still supports Ruby >= 2.6.
  • Add benchmark as a development dependency: RuboCop 1.60 requires it, and it's no longer a default gem in Ruby 4.0.
  • Add ostruct as a development dependency: the specs use OpenStruct, which is no longer a default gem in Ruby 4.0.

No runtime dependencies change other than the faraday-retry lock bump.

@khvn26
khvn26 requested a review from a team as a code owner October 29, 2026 13:32
@khvn26
khvn26 requested review from emyller and removed request for a team October 29, 2026 13:32
@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: b896bd7d-f94f-407d-ad85-35e01ea79bd1
Commits

Reviewing files that changed from the base of the PR and between f10b360 and 6001d74.

Files ignored due to path filters: 1
  • Gemfile.lock is excluded by !**/*.lock
Files selected for processing: 2
  • .github/workflows/pull_request.yaml
  • flagsmith.gemspec

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The pull request workflow adds Ruby 3.4 and 4.0 to the test matrix. The gemspec adds benchmark and ostruct as development dependencies.

Priority: Not assessed

Estimated code review effort: 2 (Simple) | ~5 minutes

Merge Risk: Minimal · up to 6001d

The PR extends Ruby CI coverage and supplies the required development gems without an identified merge-blocking issue.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Zaimwa9
Zaimwa9 previously approved these changes Oct 29, 2026
@khvn26
khvn26 changed the base branch from ci/pin-ubuntu-runner to main October 29, 2026 14:03
@khvn26
khvn26 dismissed Zaimwa9’s stale review October 29, 2026 14:03

The base branch was changed.

@khvn26
khvn26 force-pushed the ci/expand-ruby-matrix branch from 6b66d49 to 6001d74 Compare October 29, 2026 14:03
@khvn26 khvn26 changed the title ci: Test against Ruby 3.4 and 4.0 feat: Ruby 4 support Oct 29, 2026
@khvn26
khvn26 merged commit 51544ab into main Oct 29, 2026
8 of 9 checks passed
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.

2 participants