Ruby 3.4 Hash#inspect compatibility.

[voxik]

No description provided.

[radar]

Thank you! Could you please exclude Rails 6.1 + ruby-head from the matrix?

[radar]

And this combo: https://github.com/ruby-i18n/i18n/actions/runs/12235161164/job/34125870227?pr=709

[voxik]

Thank you! Could you please exclude Rails 6.1 + ruby-head from the matrix?

I have included this patch if you prefer:

diff --git a/gemfiles/Gemfile.rails-6.1.x b/gemfiles/Gemfile.rails-6.1.x
index 6f67f78..43481a6 100644
--- a/gemfiles/Gemfile.rails-6.1.x
+++ b/gemfiles/Gemfile.rails-6.1.x
@@ -8,6 +8,8 @@ gem 'test_declarative', '0.0.6'
 gem 'rake'
 gem 'minitest', '~> 5.14'
 gem 'racc'
+gem 'base64'
+gem 'mutex_m'
 
 platforms :mri do
   gem 'oj'

[voxik]

And this combo: https://github.com/ruby-i18n/i18n/actions/runs/12235161164/job/34125870227?pr=709

I'll do that. But it seems the matrix is incomplete. Rails 7.2 / 8.0 should likely be added.

[radar]

Agree. Make it so :)

…

On Wed, Dec 11, 2024, at 3:35 AM, Vít Ondruch wrote: > And this combo: https://github.com/ruby-i18n/i18n/actions/runs/12235161164/job/34125870227?pr=709 > I'll do that. But it seems the matrix is incomplete. Rails 7.2 / 8.0 should likely be added. — Reply to this email directly, view it on GitHub <#709 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAAU73UADXUJJ4PQKZVHNL2E4J3ZAVCNFSM6AAAAABTIVKC6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZSGIZTGNZZGU>. You are receiving this because you commented.Message ID: ***@***.***>

[voxik]

Agree. Make it so :)

Done.

I have noticed that I18nTest#test_uses_the_simple_backend_by_default test might be flaky. But I'm not going to investigate that.

Also, I'd say that the JRuby might need review.

[radar]

Thank you for your diligence.