Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@jjkoh95
Copy link

@jjkoh95 jjkoh95 commented Feb 3, 2023

  • Do only one thing
  • Non breaking API changes
  • Tested

What did this pull request do?

Allow default GORM tag for primary key columns

User Case Description

Refer #753

@jjkoh95
Copy link
Author

jjkoh95 commented Feb 5, 2023

This is breaking mysql 5.7, but it should never occur in the first place since they do not support expression as default. I can move tests to a separate directory dedicated for mysql 8+ if this looks good to the team.

@qqxhb
Copy link
Member

qqxhb commented Apr 11, 2023

#784

@qqxhb qqxhb closed this Apr 11, 2023
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