-
Notifications
You must be signed in to change notification settings - Fork 932
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
Automate release version generation [v8] #3268
Conversation
@@ -3,7 +3,6 @@ NODES ?= 10 | |||
PACKAGES ?= api actor command types util version integration/helpers | |||
LC_ALL = "en_US.UTF-8" | |||
|
|||
CF_BUILD_VERSION ?= $$(cat BUILD_VERSION) # TODO: version specific |
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.
question(non-blocking): do we still see some value in keeping build_data.yml up to date? As it was introduced as an eventual alternative to BUILD_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.
Good point, even that file can be removed. It was mostly used for min capi tests. Since we don't have any, will remove that as well
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.
Done
Co-authored-by: AL Berez <a-b@users.noreply.github.com>
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
While releasing CLI, take the input from the user for minor/patch bump. Based on the selection, calculate the next release version. New build version need not be stored in a file going forward.