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

Support rails 6.1 #7

Merged

Conversation

jean-francois-labbe
Copy link

@jean-francois-labbe jean-francois-labbe commented Feb 13, 2020

Fix #6

This is a working fix but may not be the best way to implement it.

Basically it restores the table_exists? Functions.

@jean-francois-labbe jean-francois-labbe changed the title Fix rails 6.1 Support rails 6.1 Feb 13, 2020
@rpbaltazar
Copy link
Contributor

rpbaltazar commented Feb 17, 2020

@jean-francois-labbe I've fixed Rubocop and CI. All should be passing now. Sorry for the merge conflict.

@rpbaltazar
Copy link
Contributor

Looking at your code, I understand where the problem comes from. I'm not sure how will this rails/rails@c5ecc33#diff-a15c218db8b9dd9f84a75d9a3894db56L30 code run. Unless I'm missing something, this almost feels like a rails bug

@rpbaltazar rpbaltazar added this to the 3.0.0 milestone Mar 24, 2020
@rpbaltazar
Copy link
Contributor

@jean-francois-labbe this is not forgotten. Just trying to clean up the development branch first and will put this under a major release

@rpbaltazar rpbaltazar merged commit 504588e into rails-on-services:development Apr 2, 2020
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.

rails 6.1.alpha support
2 participants