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

refactor: part 2 of distinguish between Unique and UniqueIndex #6822

Merged
merged 6 commits into from
Feb 6, 2024

Conversation

black-06
Copy link
Contributor

@black-06 black-06 commented Feb 5, 2024

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

What did this pull request do?

It's part 2 of #6386 which is the part of rollback, see #6386 (comment).

It will also fix the ci.

But note that it depends on

After the above requests is merged, this pr still needs to update tests/go.mod to the corresponding commit id.

User Case Description

@saeidee
Copy link
Member

saeidee commented Feb 5, 2024

To unblock new PRs, can we just resolve the CI tests issue in a dedicated PR and then decide/proceed on merging this new feature?

@black-06
Copy link
Contributor Author

black-06 commented Feb 6, 2024

I have tested this mr with the above dependency, and it is OK.

Then we need to merge the driver mrs , and update test/go.mod here.

@jinzhu

If all goes well, we can solve all the troubles today 😭

@black-06
Copy link
Contributor Author

black-06 commented Feb 6, 2024

My God, I tested it as test/docker-composite.yml, and it missed mysql 5.6

@jinzhu
Copy link
Member

jinzhu commented Feb 6, 2024

My God, I tested it as test/docker-composite.yml, and it missed mysql 5.6

This is the benefit of our CI, to catch oversights in our daily development ;)

@black-06
Copy link
Contributor Author

black-06 commented Feb 6, 2024

What a thrilling day, now we have fixed it 😭

@jinzhu jinzhu merged commit 8fb9a31 into go-gorm:master Feb 6, 2024
34 checks passed
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

3 participants