Skip to content

Conversation

@warrenvw
Copy link
Contributor

Since st2 3.0.0 has been released, bump appVersion to 3.1dev.

@warrenvw warrenvw requested a review from arm4b May 10, 2019 21:26
@warrenvw warrenvw self-assigned this May 10, 2019
## In Development

## v0.13.1
* Pin st2 version to `v3.1dev` as a new latest development version (#67)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think 0.14.0 would be more appropriate versioning since it's not a bugfix or patch but a version change.

https://github.com/StackStorm/stackstorm-ha/blob/master/CHANGELOG.md#v083 and https://github.com/StackStorm/stackstorm-ha/blob/master/CHANGELOG.md#v083 were a bit messy due to updated release plans changes and version renames.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was going to do a minor version bump for the reasons you mention, but I noticed that v0.8.4 just bumped patch when changing appVersion from v2.10dev to 3.0dev. Figured it set a precedent...

Anyhow, I agree that we shouldn't bump patch when appVersion is changed. I'll make the change you suggest.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@warrenvw warrenvw merged commit cd86771 into master May 13, 2019
@warrenvw warrenvw deleted the bump-st2-version branch May 13, 2019 18:11
@cognifloyd cognifloyd removed the RFR label Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants