Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

Bugfix for wrong handling of major only version hints. #7

Merged
merged 3 commits into from
May 31, 2018
Merged

Bugfix for wrong handling of major only version hints. #7

merged 3 commits into from
May 31, 2018

Conversation

aalbretsen
Copy link
Contributor

For a given repository with two tags "v1.0.0" and "v2.0.0" and a version hint of "1", it would have suggested "v2.1.0" as the next version instead of "v1.1.0" as expected. This is now fixed, see test code for more details.

For a given repository with two tags "v1.0.0" and "v2.0.0" and a version hint of "1", it would have suggested "v2.1.0" as the next version instead of "v1.1.0" as expected. This is now fixed, see test code for more details.
@bentsolheim
Copy link
Contributor

Dette ser veldig bra ut!
I forhold til diskusjonen vi hadde før oppgaven ble påbegynt, burde vi nå fjerne all bruk av -SNAPSHOT i dette biblioteket (og i testene) og stramme inn parsereglene for versionHint? (At det nå ikke kan inneholde noe annet enn X eller X.Y ?)

@aalbretsen
Copy link
Contributor Author

Jeg ser det som en fordel at den er mer tilgivende. Liker prinsippet om at den spiser det meste av versjoner, ser forskjell på semantiske og ikke-semantiske (snapshot) versjoner, og kan lage versjons hint av forskjellige typer ikke-semantiske versjoner ved å trekke ut de semantisk delene av hintet.

Vi har nå en bakover kompatibel løsning, spesielt med tanke på Maven plugin som vi ikke trenger å lage en ny release av.

@bentsolheim bentsolheim merged commit 3286e32 into Skatteetaten:master May 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants