-
Notifications
You must be signed in to change notification settings - Fork 25
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 support for citext arrays #43
Conversation
Can you add some test as well, please? And a changelog entry. |
Will do! |
The new specs pass locally, but i'm afraid in order to enable the |
@eneagoe I assume, CI could be fixed if you run application as superuser. I ran through tests.yml configuration file: It might be two things:
But it's just my wild guess. |
Try configuring the citext extension upfront
It looks a bit uglier now, but configuring the |
CI is working. But what would happen locally now, if I don't have citext enabled? |
Good point - I've added a condition to enable the |
@eneagoe thanks for contribution -- seems to be working locally for me too. I'll merge this and release a new gem. |
@skatkov Thank you for your time and help! |
Currently if the tags column is of
citext
type, the generated SQL is invalid. Adding this extra configuration fixes the issue.