-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Fix CHANGELOG for #15054 #15150
Fix CHANGELOG for #15054 #15150
Conversation
Signed-off-by: Jay Deng <jayd0104@gmail.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.
Ah, my bad! Missed it while reviewing #15054!
@jed326 We actually have enforcement that if you add an entry in CHANGELOG.md but do not have a backport label then this check fails. I think we could add enforcement for this case: if there is an entry in CHANGELOG-3.0.md then there must not be a backport label. Any interest in adding that check? |
@andrross sure I can take that up in a day or two, good callout! |
Signed-off-by: Jay Deng <jayd0104@gmail.com>
Signed-off-by: Jay Deng <jayd0104@gmail.com>
Signed-off-by: Jay Deng <jayd0104@gmail.com>
Description
changelog item for #15054 should not be in
CHANGELOG-3.0.md
as it was backported to2.x
in #15129Check List
- [ ] Functionality includes testing.- [ ] API changes companion pull request created, if applicable.- [ ] Public documentation issue/PR created, if applicable.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.