-
-
Notifications
You must be signed in to change notification settings - Fork 161
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
Drop support for Ruby 2.5 #836
Conversation
7b5ce29
to
f3539c4
Compare
f3539c4
to
e2dbead
Compare
I'm going to delay this until cucumber itself drops support for 2.5. |
e2dbead
to
4bc7347
Compare
Cucumber 8 has dropped support for Ruby 2.5. I'm first going to release a version with cucumber 8 support. After that, I'm going to revisit this pull request. |
b58ed24
to
c4e472c
Compare
It seems there's a consistent test failure with Ruby 3.1 on Windows that is only now coming to light because this pull request adds Ruby 3.1 to the matrix on Windows. The failure messages point to ChildProcess, but that project is not up-to-date on test environments so it's hard to tell whether Aruba is doing something wrong that only comes to light in that combination, or there's a problem with the combination of ChildProcess, Ruby 3.1 and Windows. |
- Require Ruby 2.6 or higher in the gemspec - Update RuboCop configuration and fix new offenses - Update set of Rubies in CI
ChildProcess does not play nice with Ruby 3.1 build for Windows used in GitHub Actions. See enkessler/childprocess#182
c4e472c
to
cb53cc8
Compare
I dropped 3.1 from the Windows build set for now. |
Summary
Drop support for Ruby 2.5
Details
Motivation and Context
With the release of JRuby 9.3.0.0, we can drop compatibility with Ruby 2.5. MRI 2.5 has been unmaintained for a while now.
How Has This Been Tested?
CI
Types of changes
This is a potentially breaking change for versions of rubygems and bundler that do not take ruby version requirements into account. From the standpoint of semver, it is not a breaking API change so does not require a major version bump.
Checklist: