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

Bring Ruby 2.7 support back #1107

Merged
merged 5 commits into from
Aug 30, 2024
Merged

Bring Ruby 2.7 support back #1107

merged 5 commits into from
Aug 30, 2024

Conversation

amatsuda
Copy link
Member

This PR brings Ruby 2.7 back to the CI matrix by monkey-patching Capybara to correctly switch three all methods defined by Capybara, RSpec, and Aruba.

See teamcapybara/capybara#2771 for the details of the Capybara patch.

@tycooon
Copy link

tycooon commented Aug 30, 2024

Ruby 2.7 reached EOL on 2023-03-31, why keep supporting it?

@amatsuda
Copy link
Member Author

amatsuda commented Aug 30, 2024

@tycooon Thank you for the question.

Ruby 2.7 reached EOL on 2023-03-31, why keep supporting it?

IMO testing frameworks and testing tools have to offer long enough support term for real users working on real applications.

It is true that the Ruby team currently cannot support each release no longer than three years, due to lack of maintainer resource, but I know that there are so many real world Ruby applications still running on Ruby 2.x. In fact, I as a Rails consultant used to work on a Ruby 2.4 app until a few months ago, and I'm currently still working on another Ruby 2.7 app.
And for such users, being able to run same set of tests on same set of testing tools is a lifeline for upgrading.

That's the motivation that I want to provide longer support, at least than Ruby/Rails EOL on testing tools (e.g. test-unit/test-unit-rails#32

@amatsuda amatsuda merged commit 5cf051f into simplecov-ruby:main Aug 30, 2024
7 checks passed
@amatsuda amatsuda deleted the ruby27 branch August 30, 2024 19:30
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