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

Check for add column using references #228

Merged

Conversation

adamrdavid
Copy link
Contributor

Noticed a case that squeaks by this check. There are so many ways to add a constraint in SQL 😄

ALTER TABLE "emails" ADD COLUMN "user_id" INT REFERENCES "user" ("id");

@netlify
Copy link

netlify bot commented Jul 14, 2022

👷 Deploy request for squawkhq pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 6b2709d

@sbdchd
Copy link
Owner

sbdchd commented Jul 14, 2022

Nice thank you!

@sbdchd sbdchd added the automerge automerge with kodiak label Jul 14, 2022
@kodiakhq kodiakhq bot merged commit e4e2b42 into sbdchd:master Jul 14, 2022
@adamrdavid adamrdavid deleted the check-for-add-column-using-references branch July 14, 2022 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge automerge with kodiak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants