Skip to content

Modernize supported Ruby CI versions - #426

Merged
pedro-stanaka merged 2 commits into
mainfrom
pedro-stanaka/modernize-ruby-ci
Aug 18, 2026
Merged

Modernize supported Ruby CI versions#426
pedro-stanaka merged 2 commits into
mainfrom
pedro-stanaka/modernize-ruby-ci

Conversation

@pedro-stanaka

Copy link
Copy Markdown
Contributor

Why

The CI matrix still includes Ruby 2.7, which has been EOL for a long time, while omitting Ruby 3.4 and using stale JRuby/TruffleRuby versions. The JRuby job also fails on an allocation-counting test because JRuby does not support GC.stat(:total_allocated_objects).

This is a standalone CI maintenance PR so feature PRs do not need to carry runtime-specific workarounds.

What changed

  • Drop Ruby 2.7 from the test matrix.
  • Add Ruby 3.4.
  • Update JRuby from 9.4.9.0 to 10.1.1.0.
  • Update TruffleRuby from 25.0.0 to 34.0.1.
  • Skip only test_cache_key_computation_does_not_allocate on JRuby, preserving the allocation test on CRuby and other runtimes.

Validation

  • bundle exec ruby -Itest test/compiled_metric_test.rb
  • bundle exec rake test — 376 runs, 1,061 assertions, 0 failures/errors
  • bundle exec rubocop test/compiled_metric_test.rb
  • GitHub Actions workflow YAML parses successfully

This is intended to unblock and clean up statsd-instrument PR #423, which can drop its temporary JRuby skip commit and rebase after this PR lands.

pedro-stanaka and others added 2 commits August 18, 2026 10:59
Co-authored-by: AI (Pi/GPT-5.6 Luna) <noreply@pi.dev>
Assisted-By: devx/a14ba2d6-40ac-4f2e-96da-16e215c96b8e
Co-authored-by: AI (Pi/GPT-5.6 Luna) <noreply@pi.dev>
Assisted-By: devx/a14ba2d6-40ac-4f2e-96da-16e215c96b8e
@pedro-stanaka
pedro-stanaka marked this pull request as ready for review August 18, 2026 09:08
@pedro-stanaka
pedro-stanaka merged commit 3bb30a3 into main Aug 18, 2026
11 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