Skip to content

Bump CI to JRuby 10 - #20

Merged
tungleduyxyz merged 2 commits into
mainfrom
jruby10-upgrade
Sep 9, 2026
Merged

tungleduyxyz merged 2 commits into
mainfrom
jruby10-upgrade

Conversation

@tungleduyxyz

Copy link
Copy Markdown
Contributor

Part of Phase 1 of the JRuby 10 upgrade (Rails stays ~> 7.2 for now). Bumps CI from jruby-9.4.15.0 to jruby-10.0.6.0 (LTS line). Validated locally: bundle install resolves cleanly under real JRuby 10.1.1.0/JDK21.

Bump jruby-9.4.15.0 -> jruby-10.0.6.0 (LTS line) in the CI matrix. Part of Phase 1 of the JRuby 10 upgrade (Rails stays ~> 7.2 for now). Validated locally: bundle install resolves cleanly under real JRuby 10.1.1.0/JDK21.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Defense-in-depth fix, part of the same root-cause investigation as the
CI failures fixed in killbill-kpm-ui and killbill-email-notifications-ui
(see their jruby10-upgrade branch commits for full details).

This repo currently resolves json 2.x thanks only to an INCIDENTAL
transitive constraint (rubocop's own `json ~> 2.3` dependency in whatever
version bundler happens to resolve here) - not a real guarantee. rubocop
1.90.0 (released 2026-08-24) loosened its own json dependency to
`json >= 2.3` (no upper bound), so this repo's unpinned `gem 'json'`
could silently start resolving json 3.x on any future bundle install
(Gemfile.lock isn't committed in this repo), exactly as already happened
in killbill-kpm-ui and killbill-email-notifications-ui once json
3.0.0/3.0.1 were published (2026-09-07/08).

json 3.0 dropped the quirks_mode keyword that ActiveSupport::JSON.encode
still passes to JSON.generate (not fixed until Rails 8.1.0), raising
ArgumentError on any #to_json call - including killbill-client's model
classes, used throughout this engine.

Pinned `gem 'json', '~> 2.21'` explicitly rather than relying on rubocop's
transitive constraint.

Verified: bundle install resolves json (2.21.2); test suite passes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@tungleduyxyz
tungleduyxyz merged commit 34be5d6 into main Sep 9, 2026
7 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