Skip to content

CI: Add Ruby 4.0 and Rails 8.1 to CI Matrix#243

Merged
Zogoo merged 1 commit intosaml-idp:masterfrom
taketo1113:ci-ruby4.0-rails8.1
Feb 2, 2026
Merged

CI: Add Ruby 4.0 and Rails 8.1 to CI Matrix#243
Zogoo merged 1 commit intosaml-idp:masterfrom
taketo1113:ci-ruby4.0-rails8.1

Conversation

@taketo1113
Copy link
Contributor

This Pull Request adds Ruby 4.0 and Rails 8.1 to the CI Matrix.

It also includes the following changes:

  • Fix a load error for the benchmark gem on Rails 6.1 with Ruby 4.0
  • Remove Ruby 3.2 from CI Matrix following the rails-dev (Rails 8.2) support drop
  • Bump actions/checkout from 4 to 6

Additional Information

Fix Load Error of benchmark for Rails 6.1 with Ruby 4.0

In Ruby 4.0, benchmark gem was changed from a default gem to a bundled gem, which causes the following load error on Rails 6.1.
To fix this issue, the benchmark gem has been added as a dependency for the Rails 6.1 appraisal.

An error occurred while loading ./spec/lib/saml_idp/name_id_formatter_spec.rb.
Failure/Error: Bundler.require(*Rails.groups(:assets => %w(development test)))

Bundler::GemRequireError:
  There was an error while trying to load the gem 'saml_idp'.
  Gem Load Error is: cannot load such file -- benchmark
  Backtrace for gem load error is:
  /Users/taketo/.rbenv/versions/4.0.0/lib/ruby/4.0.0/bundled_gems.rb:60:in 'Kernel.require'

CI: Drop support for Ruby 3.2 rails-dev (rails 8.2)

Rails 8.2 will drop support for Ruby 3.2, so Ruby 3.2 has been removed from the rails-dev CI matrix.

Fix a load error for the benchmark gem on Rails 6.1 with Ruby 4.0
Remove Ruby 3.2 from CI Matrix following rails-dev (Rails 8.2) support drop
Bump actions/checkout from 4 to 6
@Zogoo Zogoo self-requested a review January 11, 2026 22:06
@Zogoo Zogoo merged commit ed7e2d2 into saml-idp:master Feb 2, 2026
42 checks passed
@taketo1113 taketo1113 deleted the ci-ruby4.0-rails8.1 branch February 2, 2026 12:33
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