updated app version to 2024.4 #374
Conversation
🦋 Changeset detectedLatest commit: 1a775e5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
You will need a changeset to trigger the versioning and release process |
There was a problem hiding this comment.
The change itself is good, but needs a changeset as @APErebus mentioned.
There was a problem hiding this comment.
@liam-mackie If people install this new major version, will this upgrade their server version? I assume we use the app version for the tag, so it'll upgrade them?
There was a problem hiding this comment.
typically we use pnpm/npm... Not sure if we want to have yarn as well?
| @@ -0,0 +1,5 @@ | |||
| --- | |||
| "octopus-deploy": major | |||
There was a problem hiding this comment.
I'm ok with this. @liam-mackie ?
There was a problem hiding this comment.
This should be a minor bump IMO, as there is nothing incompatible with the current version.
| "octopus-deploy": major | ||
| --- | ||
|
|
||
| Bumped octopus server version |
There was a problem hiding this comment.
Nit: can we make this more descriptive, something like Updated installed Octopus Server version to 2024.4`.
Yes, if people install this new version, it will upgrade their server version. I don't think this is a bad thing, and we have previously updated the version this chart installs. |
|
replaced by #375 |
|
Closing as it's been replaced by #375 |
No description provided.