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 #275

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

murajun1978
Copy link

Support Ruby 3.4

Fixed #273

@Laykou
Copy link

Laykou commented Jan 1, 2025

@abrom Could this be merged and released in rubygems please? Thank you!

@@ -16,7 +16,7 @@ Gem::Specification.new do |spec|
SUMMARY
spec.homepage = 'https://github.com/Studiosity/grover'
spec.license = 'MIT'
spec.required_ruby_version = ['>= 3.0.0', '< 3.4.0']
spec.required_ruby_version = ['>= 3.0.0', '< 3.5.0']
Copy link

Choose a reason for hiding this comment

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

@abrom Any particular reason to lock Ruby version so tightly in this gem?

It'll be the same problem a few months later when 3.5 is released.
The gem prevents apps using it from upgrading Ruby.

Copy link

Choose a reason for hiding this comment

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

I agree, it is not a reasonable restriction, except in the school of enterprisey business decisions. Please relegate it to the foibles of the Ruby ecosystem's past!
This is blocking upgrades for me, so I may switch to sourcing from a git fork.

@erichstark
Copy link

Any updates on this?

Copy link

@pboling pboling left a comment

Choose a reason for hiding this comment

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

Do not restrict to less than ruby 3.5 unless you want this gem to once again be a major upgrade blocker in 11 months.

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.

Ruby 3.4 error
5 participants