Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Drop support for Rails 4.2 #251

Closed

Conversation

cjlarose
Copy link
Member

Rails 4.2 is no longer officially maintained. This PR updates the README to make clear that Rails 4.2 is not supported by config and also drops Rails 4.2 from the test suite.

Rails 4.2 is [no longer officially maintained][0]. Users of Rails 4.2
are still of course welcome to use older versions of this gem.

[0]: https://guides.rubyonrails.org/maintenance_policy.html
@pkuczynski
Copy link
Member

The current version of the gem still works well with Rails 4.2, so I am not sure if its worth going hardcore and dropping support for it? #256 should fix failing tests. Rests should work, right?

@cjlarose
Copy link
Member Author

cjlarose commented Jan 3, 2020

Yeah, if it's not a burden to keep support for Rails 4.2 in the codebase, then there's not a really compelling reason to drop support. That said, Rails 4.x users can always just install an older version.

@rdubya
Copy link
Contributor

rdubya commented Jan 4, 2020

We still use this with a 4.2 app, so I'd vote for keeping it around unless it becomes too much of a pain to deal with.

@pkuczynski
Copy link
Member

Rails 4.2 does not seem to require any extra changes in the code and everything seems to work smoothly, so there is no reason to drop it and cut people off from latest improvements.

Thanks for your PR anyway!

@pkuczynski pkuczynski closed this Jan 4, 2020
@pkuczynski pkuczynski self-requested a review January 4, 2020 12:35
@pkuczynski pkuczynski added this to the 2.2.0 milestone Jan 4, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants