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

Add semver checks #134

Merged
merged 1 commit into from
Apr 7, 2022
Merged

Add semver checks #134

merged 1 commit into from
Apr 7, 2022

Conversation

gcemaj
Copy link

@gcemaj gcemaj commented Apr 5, 2022

Use semver for version checks instead of strings

Closes: #132

build.go Outdated Show resolved Hide resolved
build.go Outdated Show resolved Hide resolved
build.go Outdated Show resolved Hide resolved
@gcemaj gcemaj force-pushed the gcemaj-semver branch 2 times, most recently from d80d50e to b360265 Compare April 5, 2022 18:17
@samj1912 samj1912 added type:task A general task semver:patch A change requiring a patch version bump labels Apr 5, 2022
build.go Outdated Show resolved Hide resolved
build.go Outdated Show resolved Hide resolved
build.go Show resolved Hide resolved
@gcemaj gcemaj force-pushed the gcemaj-semver branch 2 times, most recently from afdf252 to e876a15 Compare April 5, 2022 19:12
@gcemaj gcemaj requested a review from samj1912 April 5, 2022 19:14
@samj1912
Copy link
Member

samj1912 commented Apr 5, 2022

@gcemaj could you rebase and update the PR?

build.go Show resolved Hide resolved
build.go Outdated Show resolved Hide resolved
build.go Outdated Show resolved Hide resolved
Signed-off-by: gcemaj <gcemaj@bloomberg.net>
@samj1912 samj1912 merged commit 963d4be into buildpacks:main Apr 7, 2022
@gcemaj gcemaj deleted the gcemaj-semver branch April 8, 2022 12:31
This was referenced May 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch A change requiring a patch version bump type:task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use semvar for version checking
2 participants