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
Version 2.2.1+ not allowing remove_column to contain 4 arguments #122
Comments
Hey, I've proposed a solution. Someone from Doctolib has to review it to be merged tho I'm not surprised you didn't find the commit, it's quite hidden. I believe it was introduced because I added remove_column to the list of retriable statements, which is a one line change and is quite hidden. |
Awesome, thanks for the update 👍 |
Looks like it's solved by the latest version btw. I had to remove my modifications in the PR, but the test passes now. |
Steps to reproduce:
safety_assured
and within the block call a remove_column with 4 arguments.e.g:
the 4th arguments is a hash containing null: false and default: false
error:
I've been looking at recent commits that were supplied, but wasn't able to find in which one this issue was introduced.
The text was updated successfully, but these errors were encountered: