Skip to content
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

Add Ruby 3.4 to CI matrix #57

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add Ruby 3.4 to CI matrix #57

wants to merge 5 commits into from

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

The mongo gem hasn't released a version that supports Ruby 3.4 officially

mongo v2.22.0 would support Ruby 3.4, but it will take more time until it is released.
However, the latest version should work.
mongodb/mongo-ruby-driver#2923

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.

  • base64 - the mongo gem depends on it implicitly until 2.22.0, which has not been released yet as I wrote above
  • 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 added 3 commits March 5, 2025 01:15
the mongo gem depends on the base64 gem, which has been extracted as a bundled gem since Ruby 3.4.
Let's install the base64 gem only for test.
the activesupport gem depends on the base64 gem, which has been extracted as a bundled gem since Ruby 3.4.
Let's install the base64 gem only for test.
@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!

@a2ikm a2ikm requested a review from smtlaissezfaire March 4, 2025 17:04
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.

1 participant