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

Avoid type update in modify/3 #12

Merged
merged 2 commits into from
Nov 9, 2023
Merged

Avoid type update in modify/3 #12

merged 2 commits into from
Nov 9, 2023

Conversation

dbernheisel
Copy link
Collaborator

Add a note about avoiding modify/3 when updating a constraint on a column.

Resolve #10

@dbernheisel
Copy link
Collaborator Author

@dhedlund Let me know what you think

@dbernheisel dbernheisel merged commit af935d0 into main Nov 9, 2023
@dbernheisel dbernheisel deleted the db-avoid-type-update branch November 14, 2023 16:21
@dhedlund
Copy link

Sorry, missed the request for feedback in the PR. Looks good. It was merged to main a few days ago, but I don't see the changes on the site yet. Are there additional steps needed, or did a publish automation fail?

@dbernheisel
Copy link
Collaborator Author

The fly.io-published site does not sync, you should see it reflect on this repo however. I understand that's not optimal, I'll look into seeing if syncing is reasonably possible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants