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

feat: parses Liquibase local build DEV version #348

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

filipelautert
Copy link
Collaborator

@filipelautert filipelautert commented Mar 7, 2024

feat: when building liquibase locally version is set to DEV instead of the expected MAJOR.MINOR.PATCH pattern -> https://github.com/liquibase/liquibase/blob/1b8546371a2a29aee5d25286bbf111ed3b544d0a/liquibase-standard/src/main/resources/liquibase.build.properties#L7

This PR ensures that lpm believes this is the latest version ever.

@filipelautert filipelautert requested a review from mcred March 7, 2024 13:46
@filipelautert filipelautert self-assigned this May 29, 2024
@filipelautert filipelautert changed the title fix: parses Liquibase local build DEV version feat: parses Liquibase local build DEV version May 29, 2024
…re that lpm believes this is the latest version ever.
Copy link

sonarcloud bot commented Jul 11, 2024

@sayaliM0412
Copy link
Contributor

@filipelautert : is this still a valid PR? If yes, can we merge it?

Copy link

sonarcloud bot commented Oct 10, 2024

@filipelautert
Copy link
Collaborator Author

@filipelautert : is this still a valid PR? If yes, can we merge it?

@sayaliM0412 still a valid PR. IMHO it can be merged.

@mcred
Copy link
Contributor

mcred commented Oct 10, 2024

I'm not a fan of adding this logic to lpm. I'm not sure we should support this use case.

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.

4 participants