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 enum columns to SQlite3 dialect #2055

Merged
merged 1 commit into from May 10, 2017

Conversation

Projects
None yet
2 participants
@carragom
Contributor

carragom commented May 10, 2017

Adds support for enum columns in SQLite using a check constraint. Based on the current PostgreSQL implementation. If I missed anything please let me know.

@elhigu

This comment has been minimized.

Collaborator

elhigu commented May 10, 2017

Integration test for enum's check constraint would be great to have to actually verify that created constraints function as expected. Otherwise this looks good.

@elhigu

This comment has been minimized.

Collaborator

elhigu commented May 10, 2017

And thanks :)

@carragom

This comment has been minimized.

Contributor

carragom commented May 10, 2017

Ok sorry about that, tests are passing now. Does it look good to you or is there anything else I missed?

@elhigu

This comment has been minimized.

Collaborator

elhigu commented May 10, 2017

This is good. Just need to remember that this is actually breaking change.

@elhigu elhigu merged commit 18581f9 into tgriesser:master May 10, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment