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

Setup appraisal for rails 4 #1142

Conversation

krzysiek1507
Copy link
Contributor

No description provided.

Copy link
Collaborator

@MaicolBen MaicolBen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top work!

@MaicolBen MaicolBen requested review from Evan-M and zachfeldman April 13, 2018 19:27
Copy link
Collaborator

@Evan-M Evan-M left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@Evan-M
Copy link
Collaborator

Evan-M commented Apr 18, 2018

This is probably a completely separate issue from this PR, but Ruby 2.2.10 was the last expected release of the 2.2 series. The EOL date for Ruby 2.2 was scheduled for 3/31/2018 (2.5 weeks ago). Removing 2.2.8 from the .travis.yml file will cut out 25% of the current CI builds. (Each appraisal accounts for 9 builds atm)

I guess the bigger question is: how long should this gem support Ruby 2.2, given that it is EOL?

@MaicolBen MaicolBen merged commit ed1353f into lynndylanhurley:master Apr 19, 2018
@MaicolBen
Copy link
Collaborator

@Evan-M 2.1 and 2.2 are still widely used. I think we can keep supporting ruby 2.2 until we bump into a problematic PR that cannot be fixed without removing it (or a ruby method that increases the performance)

@krzysiek1507 krzysiek1507 deleted the maintenance/setup-appraisal-for-rails-4 branch April 23, 2018 19:29
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.

3 participants