-
Notifications
You must be signed in to change notification settings - Fork 30
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 github action to release tempto #286
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
41c1ff2
to
94fb52b
Compare
major=$(echo $release_version | cut -d. -f1) | ||
minor=$(echo $release_version | cut -d. -f2) | ||
# Increment the minor version | ||
last_minor=$((minor - 1)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what happens here if minor == 0? e.g. assume we release a 2.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, seems I need a new parameter and flag for this, I'll update later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think you need to add a flag -- the likelihood that we ever end up releasing anything other than 0.x is low. However, I think it would be good to add a check and just error out here with a message saying that the pipeline doesn't handle a new major version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated @ZacBlanco
This PR is sitll in progress.
Testing action in this repo => https://github.com/unix280/tempto/actions/workflows/release.yml
Test passed with test GPG key and test maven repo => https://github.com/unix280/tempto/actions/runs/13102076676
Environment setup:
release
release
environment settings