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

Extend Liquibase IT with includeAll #16293

Merged
merged 2 commits into from
Apr 6, 2021
Merged

Extend Liquibase IT with includeAll #16293

merged 2 commits into from
Apr 6, 2021

Conversation

famod
Copy link
Member

@famod famod commented Apr 6, 2021

Resolves #14751 by verifying that Liquibase upgrade in #15972 fixed it (in JVM mode)

famod added 2 commits April 6, 2021 19:35
Resolves #14751 by verifying that Liquibase upgrade in #15972 fixed (it in JVM mode)
@gastaldi
Copy link
Contributor

gastaldi commented Apr 6, 2021

Just not sure if it needs to be backported, but looks ok

@gastaldi gastaldi merged commit c801723 into quarkusio:main Apr 6, 2021
@quarkus-bot quarkus-bot bot added this to the 1.14 - main milestone Apr 6, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Apr 6, 2021

Milestone is already set for some of the items:

We haven't automatically updated the milestones for these items.

This message is automatically generated by a bot.

@famod famod deleted the liquibase-includeAll branch April 6, 2021 20:38
@famod
Copy link
Member Author

famod commented Apr 6, 2021

I propose to backport this to be consistent with #15972 (which already ended up in the 1.13 branch for 1.13.1).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Liquibase includeall only works in dev mode
3 participants