-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Release 15.0.2 #38315
Release 15.0.2 #38315
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.
The last 2 bullets don't require a minor version bump. We don't strictly adhere to semver as it would require us to hold features back and we prefer to ship things when they're ready.
I agree that the changelog entry is not clear. We need to describe what action is required of users.
@EdwardDowling can you please clarify what action is required by end users? |
@fheinecke I disabled automerge because there's still some open question about the changelog, not sure if you'd rather fix it with a later PR. |
@fheinecke |
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.
@fheinecke @EdwardDowling Let's hold off for a bit. It seems to me that the Opsgenie fix introduces a backwards incompatible behavior which we should not do in a patch release.
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.
@fheinecke Let's rebase, remove the Opsgenie changelog line and publish.
Reviewers, please take a look at the below items specifically:
Changelog:
tsh status
command. #38305