Skip to content

Stabilize JRuby 9.2 Rails 5.2 matrix setup by pinning Bundler for JRuby Ruby 2.5#1640

Merged
estolfo merged 3 commits into
mainfrom
copilot/fix-matrix-test-jruby-9-2-rails-5-2
Jun 19, 2026
Merged

Stabilize JRuby 9.2 Rails 5.2 matrix setup by pinning Bundler for JRuby Ruby 2.5#1640
estolfo merged 3 commits into
mainfrom
copilot/fix-matrix-test-jruby-9-2-rails-5-2

Conversation

Copilot AI commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

The matrix / test (jruby:9.2, rails-5.2) job was failing during dependency resolution, crashing inside Bundler with Bundler::Settings::Mirrors::Mirror under the JRuby 9.2 runtime path. This change isolates that compatibility path without altering MRI behavior.

  • Root cause alignment

    • The failure occurs in the Ruby 2.4/2.5 setup branch used by JRuby 9.2, where Bundler ~>2.3.x can raise a mirror constant NameError during bundle update.
  • Targeted compatibility fix

    • In spec/build/update.sh, the Ruby 2.4/2.5 branch now selects Bundler by runtime:
      • JRuby: bundler ~> 2.2.21
      • non-JRuby: keep existing bundler ~> 2.3.26
    • This keeps existing behavior for MRI and narrows the change to the failing JRuby path.
  • Code change (excerpt)

    elif [[ "${RUBY_VERSION}" =~ ^2\.(4|5).+ ]]; then
      gem i "rubygems-update:~>2.7" --no-document
      update_rubygems --no-document
      if ruby -e 'exit defined?(JRUBY_VERSION) ? 0 : 1'; then
        gem i "bundler:~>2.2.21" --no-document
      else
        gem i "bundler:~>2.3.26" --no-document
      fi
    fi

Copilot AI changed the title [WIP] Fix failing GitHub Actions job matrix / test (jruby:9.2, rails-5.2) Stabilize JRuby 9.2 Rails 5.2 matrix setup by pinning Bundler for JRuby Ruby 2.5 Jun 19, 2026
Copilot AI requested a review from estolfo June 19, 2026 11:04
@estolfo estolfo marked this pull request as ready for review June 19, 2026 11:13
@estolfo estolfo requested a review from hegerchr June 19, 2026 12:14

@hegerchr hegerchr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@estolfo estolfo merged commit f66714c into main Jun 19, 2026
47 checks passed
@estolfo estolfo deleted the copilot/fix-matrix-test-jruby-9-2-rails-5-2 branch June 19, 2026 12:23
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.

3 participants