-
Notifications
You must be signed in to change notification settings - Fork 1.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
Fix getResourceAccessor logic to avoid setting changeLogDirectory when searchPath has already set #3347
Conversation
…when having a searchPath already set.
f2932ee
to
d57fdb8
Compare
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.
Changes look good to me
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.
This PR addresses a bug where Liquibase Maven disregarded the changeLogDirectory
property, resulting in the unexpected location of two changelog files (one in src, the other in target). Now Liquibase will look for a changelog only within the specified changeLogDirectory
.
- New unit tests added.
- No additional testing required.
APPROVED
Impact
Description
Fixes #3282
Things to be aware of
Things to worry about
Additional Context