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

Make down sql optional for the set_unique operation #119

Merged
merged 3 commits into from
Sep 22, 2023

Conversation

andrew-farries
Copy link
Collaborator

Follow up to #118 to make the down SQL optional.

When making an existing column unique, the down sql is almost always going to be a simple copy from the new column to the old.

@andrew-farries andrew-farries force-pushed the copy-and-trigger-for-set-unique-operation branch from 66c217a to 486ec86 Compare September 22, 2023 08:48
Base automatically changed from copy-and-trigger-for-set-unique-operation to main September 22, 2023 08:52
When making an existing column unique, the `down` sql is almost always
going to be simple copy from the new column to the old.
@andrew-farries andrew-farries enabled auto-merge (squash) September 22, 2023 08:55
@andrew-farries andrew-farries merged commit 2d574ff into main Sep 22, 2023
9 checks passed
@andrew-farries andrew-farries deleted the set-unique-make-down-optional branch September 22, 2023 09:00
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.

None yet

2 participants