Skip to content

Conversation

@a2ikm
Copy link
Contributor

@a2ikm a2ikm commented Dec 26, 2024

Overview

This PR adds Ruby 3.4 to the CI matrix.
We don't need to update the implementation for Ruby 3.4.

Ruby 3.4 Release Note:
https://www.ruby-lang.org/en/news/2024/12/25/ruby-3-4-0-released/

Notes

Developers who use Ruby 3.4 may need to add the some gems for implicit dependencies

Ruby 3.4 has extracted some bundled gems. plucky depends on the following ones indirectly.

  • bigdecimal - the activesupport gem depends on it implicitly until 7.0.0

I've added them to Gemfile only to pass the tests.
Developers can also fix dependency errors by adding missing gems to their Gemfile themselves.

Related

@a2ikm a2ikm changed the title Add Ruby 3.4 to CI [WIP] Add Ruby 3.4 to CI Dec 26, 2024
@a2ikm a2ikm force-pushed the ruby-3-4 branch 2 times, most recently from b7fcd86 to c4dc102 Compare March 4, 2025 16:15
@a2ikm a2ikm changed the title [WIP] Add Ruby 3.4 to CI Support Ruby 3.4 Mar 4, 2025
@a2ikm a2ikm changed the title Support Ruby 3.4 Add Ruby 3.4 to CI matrix Mar 4, 2025
@a2ikm a2ikm marked this pull request as ready for review March 4, 2025 17:04
@a2ikm
Copy link
Contributor Author

a2ikm commented Mar 4, 2025

@smtlaissezfaire
Hi 👋
Could you review this PR?
I'm concerned that we are going to announce via README that plucky supports Ruby 3.4 even though the mongo gem hasn't yet officially.

Note that we don't need to release a new plucky version to support Ruby 3.4.

Thanks!

(Updated)
The mongo gem supports Ruby 3.4 since v2.21.1.
mongodb/mongo-ruby-driver@v2.21.0...v2.21.1#diff-d6331b8e19871f5b4ca5dbbcf8f7e2d2d4955af76da4ac90fa30c7350b95bba3R40

So my concern is gone 😄

@a2ikm a2ikm requested a review from smtlaissezfaire March 4, 2025 17:04
@a2ikm a2ikm force-pushed the ruby-3-4 branch 2 times, most recently from 0bb28da to 5971460 Compare June 29, 2025 11:23
the activesupport gem depends on the bigdecimal gem implicitly until 7.0.0, which has been extracted as a bundled gem since Ruby 3.4.
Let's install the bigdecimal gem only for test.
@a2ikm
Copy link
Contributor Author

a2ikm commented Jul 2, 2025

@smtlaissezfaire
CI looks good, and we don't need to change the implementation.
Let me merge, please.

If you have any concern, please feel free to revert.

@a2ikm a2ikm merged commit f7e8203 into master Jul 2, 2025
15 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