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

UUID schema enforcer #546

Merged

Conversation

matthewmcgarvey
Copy link
Member

@matthewmcgarvey matthewmcgarvey commented Nov 25, 2020

Related to #393

If we ever wanted to support generating UUID primary keys from the database, this validation will aid in catching tables that need to be updated to set a default value. I did not add it to the list of validations that run by default.

@matthewmcgarvey matthewmcgarvey force-pushed the uuid-schema-enforcer branch 2 times, most recently from 59b152c to 7e5ea04 Compare November 25, 2020 16:29
@matthewmcgarvey matthewmcgarvey marked this pull request as ready for review November 25, 2020 16:29
Copy link
Member

@jwoertink jwoertink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I love this!

@matthewmcgarvey matthewmcgarvey merged commit e68db26 into luckyframework:master Dec 2, 2020
@matthewmcgarvey matthewmcgarvey deleted the uuid-schema-enforcer branch December 2, 2020 19:31
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.

2 participants