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

Make plugin compatible with Liquibase v4.11.0 and up #11

Merged
merged 12 commits into from
May 17, 2023
Merged

Make plugin compatible with Liquibase v4.11.0 and up #11

merged 12 commits into from
May 17, 2023

Conversation

kdeenkhoorn
Copy link
Contributor

@kdeenkhoorn kdeenkhoorn commented Feb 11, 2023

xld-liquibase-plugin updated to v5.1.0

The following changes were made:

  • Dependencies are now included in the XLDP file.
  • Support for rollbackVersionPrefix is added to provide more flexibility.
  • Support for Liquibase v4.11.0 (and up) launcher script is added, the old way of starting Liquibase is maintained for compatibility reasons.

This pull request includes changes proposed in Pull request #10

@kdeenkhoorn
Copy link
Contributor Author

Reverted stepordering commits in the plugin for now, although it works, it does have it's affect on migration scenarios. Whenever one configures steplistoptions on an existing deployment XL Deploy sees this as an update and executes the Liquibase changelog again from the beginning. It is understandable behavior but i'm not sure if it is something you want it to do.

@kdeenkhoorn
Copy link
Contributor Author

kdeenkhoorn commented Feb 13, 2023

Found and fixed a syntax error in the test.(sh/bat).ftl

Copy link

@isaacsatish isaacsatish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@kdeenkhoorn
Copy link
Contributor Author

Who will perform the merge of the pullrequest ? I don't have write access to the repo.

@Murugesan-30795 Murugesan-30795 merged commit 1b34280 into xebialabs-community:master May 17, 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.

None yet

3 participants