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 support #133

Merged
merged 2 commits into from
Feb 7, 2025
Merged

Add Ruby 3.4 support #133

merged 2 commits into from
Feb 7, 2025

Conversation

JuanVqz
Copy link
Member

@JuanVqz JuanVqz commented Feb 4, 2025

Description

Ruby 3.4 is removing the base64 default gem to became bundled

/Users/juan/.asdf/installs/ruby/3.3.5/lib/ruby/gems/3.3.0/gems/webmock-3.16.2/lib/webmock.rb:14: warning: base64 was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.4.0.
You can add base64 to your Gemfile or gemspec to silence this warning.

Making the current webmock (v3.16.2) gem incompatible because it has a dependency on the base64 gem,
to fix it we are bumping webmock to v3.20.0 which removed the
base64
dependency.

Note

As an alternative solution, we can leave the current webmock version untoched
and add the base64 gem as a dependency.

Motivation and Context

Increase the compatibility of the project with the latest Ruby versions.

How Has This Been Tested?

  • Run the test suite

Screenshots:

I will abide by the code of conduct

Ruby 3.4 is removing the `base64` default gem to [became bundled](https://rubyreferences.github.io/rubychanges/3.4.html#default-gems-that-became-bundled)

Making the current webmock (v3.16.2) gem incompatible because it has a dependency on the `base64` gem,
to fix it we are bumping webmock to v3.20.0 which [removed the
`base64`](bblimke/webmock#1046) dependency.

> [!NOTE]
> As an alternative solution, we can leave the current `webmock` version untoched
and add the `base64` gem as a dependency.
@JuanVqz JuanVqz force-pushed the add-ruby-3-4-support branch from 249076e to 816e5b6 Compare February 4, 2025 19:50
@@ -13,7 +13,7 @@ Gem::Specification.new do |spec|
spec.homepage = "https://github.com/fastruby/next_rails"
spec.license = "MIT"

spec.required_ruby_version = ">= 2.0"
spec.required_ruby_version = ">= 2.0", "< 3.5"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought it would be ideal to block supported Ruby versions to make sure users can rely on it, happy to remove it if you don't find it valuable tho

@JuanVqz JuanVqz marked this pull request as ready for review February 4, 2025 19:53
Copy link
Member

@etagwerker etagwerker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JuanVqz I like these changes but I'd like to skip the change limiting the required ruby version.

next_rails.gemspec Outdated Show resolved Hide resolved
@JuanVqz JuanVqz requested a review from etagwerker February 7, 2025 04:37
@etagwerker etagwerker merged commit 8cdd756 into main Feb 7, 2025
7 checks passed
@etagwerker etagwerker deleted the add-ruby-3-4-support branch February 7, 2025 13:56
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