Skip to content

Drop support to EOL rubies and Rails#5600

Merged
carlosantoniodasilva merged 6 commits intomainfrom
rm-drop-support-to-old-rubies-and-rails
Oct 13, 2023
Merged

Drop support to EOL rubies and Rails#5600
carlosantoniodasilva merged 6 commits intomainfrom
rm-drop-support-to-old-rubies-and-rails

Conversation

@rafaelfranca
Copy link
Collaborator

@carlosantoniodasilva I'm not sure if you were keeping support to those old versions by design, but maybe it is time for us to let it go. All tests on Ruby < 2.7 were failing.

@rafaelfranca rafaelfranca force-pushed the rm-drop-support-to-old-rubies-and-rails branch 2 times, most recently from 71b0468 to a231897 Compare June 9, 2023 23:26
rafaelfranca and others added 4 commits October 12, 2023 16:49
All Ruby 2.x are EOL now.
It's EOL but the current versions of Rails are still supporting it, so
it makes sense for us to keep supporting it as well. And it doesn't give
us any extra maintenance burden as older versions have been doing.
@carlosantoniodasilva carlosantoniodasilva force-pushed the rm-drop-support-to-old-rubies-and-rails branch from a231897 to 72b421e Compare October 13, 2023 13:44
Also change back to run CI on ubuntu-latest which should work with all
Ruby versions supported now.
They were added to support an old/deprecated feature of Rails, as the
message was saying, we can now remove them from the code as well.
@carlosantoniodasilva
Copy link
Member

Thanks @rafaelfranca! I was hesitant to drop support and have to go to a 5x release, but it's probably time, there's a few other potential breaking changes we can then include to move things forward.

I've branched out a 4-stable to release any bug fixes as needed, so let's make main target a 5x release and drop these old versions. I added a couple extra commits, but mainly I wanted to keep support to Ruby 2.7, even though it's EOL I think it makes sense to support it for a while longer to keep better compatibility with Rails. We can drop it at a later date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants