-
Notifications
You must be signed in to change notification settings - Fork 89
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.x support #278
base: master
Are you sure you want to change the base?
Add Ruby 3.x support #278
Conversation
4034b30
to
0d4217e
Compare
15c622e
to
d4f5c91
Compare
docile (~> 1.1.0) | ||
json (~> 1.8) | ||
set (1.1.0) | ||
simplecov (0.17.1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bumped simplecov because the underlying json version for the previous version is not compatible with ruby 3.x
@@ -19,7 +19,7 @@ Gem::Specification.new do |s| | |||
s.extra_rdoc_files = ['README.md'] | |||
s.rdoc_options << '--title' << 'Datastax Ruby Driver' << '--main' << 'README.md' << '--line-numbers' | |||
|
|||
s.required_ruby_version = '>= 2.2.0' | |||
s.required_ruby_version = '>= 2.3.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorted_set requires ruby 2.3. I've seen libraries not consider bumps in minimum ruby versions breaking changes when the dropped ruby version(s) is passed EOL so I'll defer here to the maintainers w.r.t versioning.
fc391c6
to
450fddc
Compare
450fddc
to
e5fde7c
Compare
Hey @absurdfarce all the specs here are passing but it looks like
Do you want to update or drop testing this ruby version, or continue to ignore this failure? |
Adds
sorted_set
dependency so the gem can run under Ruby 3.x and updates the CI to test against Ruby 3.0 and 3.1.Fixes https://datastax-oss.atlassian.net/browse/RUBY-336.