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

Feature/semantic versions #14

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

Conversation

ShaneLee
Copy link

This PR addresses issue #10 by providing support for semantic versioning when the version number is separated by dots or underscores.

I have implemented the version number as a double whilst retaining the original version string. The one issue with this is that in the comparisons of versions 1.23 would be lower that 1.23.4 which I think is fine as I can't see people having two different versioning schemes. I can do something about it if required.

@ShaneLee ShaneLee marked this pull request as ready for review December 13, 2021 21:12
@nkonev
Copy link
Owner

nkonev commented Dec 14, 2021

Hello. I briefly looked at this PR.
Thank you for your contribution.
The main obstacle is that in the database the migration id is integer, for example https://github.com/nkonev/r2dbc-migrate/blob/1.8.0/r2dbc-migrate-core/src/main/java/name/nkonev/r2dbc/migrate/core/PostgreSqlQueries.java#L49 and given this we need to make "migration for migration". Which is not a trivial itself.
Also I would to keep this tool as simple as possible.

I am not going to merge it right now.
I think we can delay this PR for the future 2.x family.

@nkonev nkonev added this to the V2.x milestone Dec 14, 2021
@nkonev nkonev removed this from the V3.x milestone Nov 29, 2022
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