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

Test on Rails 5.2 & fix mysql for older Rails #304

Merged
merged 3 commits into from
Mar 25, 2018

Conversation

swrobel
Copy link
Contributor

@swrobel swrobel commented Mar 24, 2018

No description provided.

@swrobel swrobel force-pushed the rails-52 branch 4 times, most recently from b0f22e4 to 66e9a6c Compare March 25, 2018 04:59
@mceachen
Copy link
Collaborator

Nice!

Boy howdy is that a serious test matrix now--Travis builds look like they will go from 40 minutes (!) to over an hour (!!). I was hoping we could drop some of those ruby versions, but as of this moment, they're all currently supported. 2.2 should EOL soon.

Perhaps we should remove the tests whose results we always discard, like 2.6 and activerecord_edge?

@mceachen mceachen merged commit a27b0a1 into ClosureTree:master Mar 25, 2018
@swrobel
Copy link
Contributor Author

swrobel commented Mar 26, 2018

FYI I have an issue open w/ Travis about the 2.6 build failing travis-ci/travis-ci#9383 (comment)

@mceachen
Copy link
Collaborator

Yeah, that build took 80 minutes, so it doubled. Do you think we should include all the ruby versions? I don't remember ever seeing a difference in how 2.x rubies performed in tests (whereas there certainly was between 1.x and 2.x, of course).

@swrobel
Copy link
Contributor Author

swrobel commented Mar 26, 2018

Good point. I think it can be cut down to oldest supported version. Activrecord versions are more important.

@swrobel swrobel deleted the rails-52 branch March 26, 2018 04:52
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