Skip to content

support newer major ruby versions#15

Merged
kratob merged 2 commits intorails-lts:mainfrom
guille:main
Jan 13, 2026
Merged

support newer major ruby versions#15
kratob merged 2 commits intorails-lts:mainfrom
guille:main

Conversation

@guille
Copy link
Copy Markdown
Contributor

@guille guille commented Dec 30, 2025

Trying to require this gem from Ruby 4 breaks because the condition evaluates to false. Despite the gem name I think it's fair to extend it to "3 or newer"?

@guille
Copy link
Copy Markdown
Contributor Author

guille commented Jan 12, 2026

@kratob is this something you'd be interested in merging or is the idea that this gem shouldn't be used in Ruby 4?

@kratob kratob merged commit bb9dde6 into rails-lts:main Jan 13, 2026
5 checks passed
@kratob
Copy link
Copy Markdown
Member

kratob commented Jan 13, 2026

@guille Thanks for the PR. You're right, this should work for Ruby 4; I guess the gem is not very well-named.

Released the change as version 1.5.1.

@guille
Copy link
Copy Markdown
Contributor Author

guille commented Jan 13, 2026

Thanks! Have a nice day

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