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

BAH-3989 | Fix. Remove Script Based Liquibase Run And Bump Dependencies #115

Merged
merged 2 commits into from
Jun 27, 2024

Conversation

rahu1ramesh
Copy link
Contributor

@rahu1ramesh rahu1ramesh commented Jun 24, 2024

JIRA -> BAH-3989

In this PR, the following changes have been added:

  • Remove Dependency on Script: Eliminated the need for the run-liquibase script.
  • Automatic Execution: Configured the bean to automatically run Liquibase changesets during the application startup phase.
  • Rename Changeset XML File: Renamed the changeset XML file to avoid conflicts caused by multiple XML files.
  • Upgrade Dependencies: Upgraded the versions of both Liquibase and Postgres to address critical security vulnerabilities.

Copy link
Member

@mohan-13 mohan-13 left a comment

Choose a reason for hiding this comment

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

@rahu1ramesh The changes looks good to me. Since this is a major change on the method of execution of liquibase, hope you have validated with different scenarios like running on existing database, running on fresh database.

@rahu1ramesh rahu1ramesh requested a review from mohan-13 June 25, 2024 09:48
@rahu1ramesh rahu1ramesh changed the title BAH-3989 | Fix. Remove Script Based Liquibase Run BAH-3989 | Fix. Remove Script Based Liquibase Run And Bump Dependencies Jun 25, 2024
@mohan-13 mohan-13 merged commit ba9913f into master Jun 27, 2024
2 checks passed
@mohan-13 mohan-13 deleted the BAH-3989 branch June 27, 2024 04:17
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.

2 participants