-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Add configuration option for liquibase.allowDuplicatedChangesetIdentifiers #40603
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah nice, this approach is indeed a lot cleaner than what I had in mind.
I would not have thought of it had you not brought up the system property idea 😎 |
Actually, the first commit can be improved by using a Map instead of the custom type I added. I'll update the PR tomorrow |
Status for workflow
|
P.S. This is draft because I don't know how to test it