-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
github: require release note on pull requests #4555
Conversation
I think the release note should be changed when release, instead of a single PR? |
@breeswish I think it's okay to change the changelog on PRs, some projects also do this, eg rocksdb. Collecting release note when release is sometimes annoying and we may forget to mention important changes. |
PTAL @dcalvin |
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.
See my suggested edits in line.
Signed-off-by: Neil Shen <overvenus@gmail.com> Update .github/pull_request_template.md Co-Authored-By: overvenus <overvenus@gmail.com> Update .github/pull_request_template.md Co-Authored-By: overvenus <overvenus@gmail.com> Signed-off-by: Neil Shen <overvenus@gmail.com>
PTAL, thanks! |
/run-unit-test |
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.
LGTM
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.
LGTM
/run-unit-test |
Signed-off-by: Neil Shen <overvenus@gmail.com> Update .github/pull_request_template.md Co-Authored-By: overvenus <overvenus@gmail.com> Update .github/pull_request_template.md Co-Authored-By: overvenus <overvenus@gmail.com> Signed-off-by: Neil Shen <overvenus@gmail.com>
Signed-off-by: Neil Shen <overvenus@gmail.com> Update .github/pull_request_template.md Co-Authored-By: overvenus <overvenus@gmail.com> Update .github/pull_request_template.md Co-Authored-By: overvenus <overvenus@gmail.com> Signed-off-by: Neil Shen <overvenus@gmail.com>
What have you changed? (mandatory)
Require release note on pull requests if it should be mentioned.
What are the type of the changes? (mandatory)
How has this PR been tested? (mandatory)
No.
Does this PR affect documentation (docs) update? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.