Skip to content

Upgrade to Rails 7.2 - #26

Open
tungleduyxyz wants to merge 2 commits into
masterfrom
upgrade_rails_7.2
Open

Upgrade to Rails 7.2#26
tungleduyxyz wants to merge 2 commits into
masterfrom
upgrade_rails_7.2

Conversation

@tungleduyxyz

@tungleduyxyz tungleduyxyz commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Upgrade the engine to Rails 7.2.

  • Require rails ~> 7.2 in the gemspec
  • Require Ruby >= 3.1.0 (minimum Ruby version for Rails 7.2)

See killbill/killbill-admin-ui#655 for the corresponding kaui engine upgrade.

JRuby CI fix (i18n / Fiber.current)

CI on jruby-9.4.2.0 failed during boot with:

NoMethodError: undefined method `current' for Fiber:Class
Did you mean?  __current__
  initialize at i18n-1.15.2/lib/i18n/config.rb:14

Root cause: i18n 1.15.x switched I18n.config to fiber-based storage. The original 1.15.0 report (ruby-i18n/i18n#735) covered the Fiber[:i18n_config] call breaking CRuby < 3.2; 1.15.1/1.15.2 reworked that but I18n::Config#initialize still calls Fiber.current. JRuby 9.4.2.0 doesn't expose a public Fiber.current (only the internal __current__; the public alias was added in a later 9.4.x release), so any i18n >= 1.15.0 fails at boot on that runtime.

This PR therefore also:

  • Locks i18n to ~> 1.14.0 (1.14.x has no fiber code path, and satisfies activesupport 7.2's i18n >= 1.6, < 2) — matching the existing pins in kaui and kaui-standalone
  • Bumps CI JRuby from 9.4.2.0 to 9.4.15.0 (which provides a proper Fiber.current) — matching kaui's CI

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Upgrades the killbill-avatax mountable engine’s Rails dependency to 7.2, updates the gem’s minimum supported Ruby version accordingly, and adjusts dependency/CI settings to address JRuby boot failures related to i18n fiber storage.

Changes:

  • Update runtime dependency from rails ~> 7.0 to rails ~> 7.2 and set required_ruby_version >= 3.1.0
  • Pin i18n to ~> 1.14.0 to avoid JRuby compatibility issues with Fiber.current
  • Bump CI JRuby from 9.4.2.0 to 9.4.15.0

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
Gemfile Adds an i18n pin intended to avoid JRuby boot errors.
avatax.gemspec Upgrades Rails requirement to 7.2 and raises the minimum Ruby version.
.github/workflows/ci.yml Updates JRuby version in CI matrix to a release that provides Fiber.current.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread Gemfile
Comment on lines +10 to +12
# Lock i18n to 1.14.x for: https://github.com/ruby-i18n/i18n/issues/735
gem 'i18n', '~> 1.14.0'

Comment thread avatax.gemspec
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