-
-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix CI for Ruby3.4 #274
base: master
Are you sure you want to change the base?
Fix CI for Ruby3.4 #274
Conversation
Old minitest depends on mutex_m gem (before v5.21) without explicit dependency. So it causes dependency errors on CI because the mutex_m gem became the bundled gem since Ruby 3.4. This upgrade minitest to the latest version which does not depends on the mutex_m gem.
Since Ruby 3.4, mutex_m gem becames the bundled gem. So it should be listed on the dependencies list. Note: The latest activesupport now does not depends on it. So it's also okay to upgrade it to the new one.
Since Ruby 3.4, abbrev gem has became the bundled gem. But old rbs gem does not depend on it. This updates rbs gem to v3.4, which depends on abbrev gem explicitly.
Now I added an additional commit:
|
Since Ruby 3.4, base64, bigdecimal, and drb gems have became the bundled gem. So they should be listed on the dependencies list. Note: The latest activesupport now depends on them. So it's also okay to upgrade it to the new one.
ace4687
to
5c94b64
Compare
I also added another commit:
|
Our CI with Ruby3.4 has been broken because the mutex_m gem has become the bundled gem since 3.4.
This fixes it via modifying dependencies.
Upgrade minitest to v5.21.1 (latest)
Old minitest depends on mutex_m gem (before v5.21) without explicit
dependency. This upgrade minitest to the latest version which does not depends on
the mutex_m gem.
Add mutex_m to the dependencies of test app
Since Ruby 3.4, mutex_m gem becames the bundled gem. So it should be
listed on the dependencies list.
Note: The latest activesupport now does not depends on it. So it's also
okay to upgrade it to the new one instead of adding.