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

Add tests for Rails 5.2 #388

Merged
merged 1 commit into from Dec 30, 2017

Conversation

mrhead
Copy link
Contributor

@mrhead mrhead commented Nov 29, 2017

Rails 5.2 is imminent so we should add support for it.

@mrhead mrhead force-pushed the prepare-for-rails-5-2 branch 4 times, most recently from 91e29f5 to ddc2b03 Compare November 29, 2017 05:37
Rails 5.2 is imminent so we should add support for it.
@tbrisker
Copy link
Collaborator

tbrisker commented Dec 3, 2017

@mrhead looks like now tests for 5.1 fail on mysql, care to check why?

@tbrisker
Copy link
Collaborator

tbrisker commented Dec 3, 2017

disregard my comment, looks like #387 fixes this issue - apparently 5.2.0.beta2 < 5.2 causing the failures in CI.

@mrhead
Copy link
Contributor Author

mrhead commented Dec 3, 2017

@tbrisker Awesome :)

@mrhead
Copy link
Contributor Author

mrhead commented Dec 3, 2017

Anyway, just let me know if there is any action needed. I'll be happy to check it.

@danielmorrison danielmorrison merged commit 5b291e5 into collectiveidea:master Dec 30, 2017
@mrhead mrhead deleted the prepare-for-rails-5-2 branch March 27, 2018 14: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.

None yet

3 participants