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

Assume that a "auto_increment"-ing non-nullable column "hasDefault" #11

Merged
merged 3 commits into from
Sep 8, 2021

Conversation

maranomynet
Copy link
Collaborator

This makes sure that primary key "id" fields are flagged as optional for inserts.

BTW, this PR builds on (and thus contains) #7

maranomynet and others added 3 commits January 14, 2021 17:32
Fixes a column-name issue that seems to affect mysql@8

See description/info here: SweetIQ/schemats#97
This makes sure that primary key "id" fields are flagged as optional 
for inserts.
@nettofarah
Copy link
Owner

Hey, @maranomynet.
I just merged your other PR and tried to get this one to work, but something is causing tests to fail.

Sorry about the late response here, my github notifications are kind of messed up.

@nettofarah nettofarah merged commit 8200b94 into nettofarah:master Sep 8, 2021
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