-
Notifications
You must be signed in to change notification settings - Fork 84
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
Create a new major version of the MQTT package, to adopt MQTTnet v4.x #295
Conversation
RELEASING.md
Outdated
|
||
A single GitHub release (and tag) will be created for each beta release, to cover all packages. | ||
|
||
- Example tag name: "CloudNative.CloudEvents.All-2.0.0" | ||
- Example release title: "All packages version 2.0.0" | ||
|
||
### Exception: new major versions | ||
|
||
Sometimes, we need a new major version of a "satellite" package, typically |
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.
Sometimes, we need a new major version of a "satellite" package, typically | |
Usually, all packages shipped from this repo share the same major version. Sometimes, we need a new major version of a "satellite" package, typically |
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.
Hmm... I think I'll reword again. (Because it's a "usually in the sense of most packages" rather than a "usually in the sense of most releases".) Thanks for pointing out that it needs tweaking :)
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've reworded this whole paragraph and added the specific MQTT example (in the past tense, to avoid having to update it again after the release I expect to do as soon as this is in).
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.
Rewording looks good! I mostly wanted to make sure we documented that a different major version was the exception, not the norm, with our current versioning strategy. I think your latest delta conveys that sufficiently.
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! I really appreciate the attention to detail :) Will merge now, then create a release PR (possibly tomorrow).
85e9414
to
e9cd37b
Compare
This more fine-grained way of referring to the versions will allow us to still only change Directory.Build.props just before a release, but have separate major versions flow from that. Signed-off-by: Jon Skeet <jonskeet@google.com>
This involves a new major version of CloudNative.CloudEvents.Mqtt. Signed-off-by: Jon Skeet <jonskeet@google.com>
Fixes #269.
cc @captainsafia, @antiblue, @Clockwork-Muse
This PR doesn't create the new release - I'd do that in a separate (tiny) PR.