-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
chore: update PR template to suggest cherry-pick releases #16560
chore: update PR template to suggest cherry-pick releases #16560
Conversation
Signed-off-by: Jesse Suen <[email protected]>
@ericblackburn - this is what I proposed in today's meeting |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #16560 +/- ##
=======================================
Coverage 49.48% 49.49%
=======================================
Files 270 270
Lines 47489 47489
=======================================
+ Hits 23502 23506 +4
+ Misses 21675 21672 -3
+ Partials 2312 2311 -1 ☔ View full report in Codecov by Sentry. |
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, thanks Jessie!
…6560) Signed-off-by: Jesse Suen <[email protected]>
…6560) Signed-off-by: Jesse Suen <[email protected]>
…6560) Signed-off-by: Jesse Suen <[email protected]>
…6560) Signed-off-by: Jesse Suen <[email protected]> Signed-off-by: Kevin Lyda <[email protected]>
…6560) Signed-off-by: Jesse Suen <[email protected]>
In today's community meeting, I proposed that the authors of PRs could suggest how far back a bug fix should be cherry-picked. This adds an item as part of the checklist.
This allows the author to advocate for a cherry-pick, since they are fixing the bug for a reason (it probably impacts them). The fix may or may not be backported depending on the risk/complexity of the fix, as well as the age of the release, as decided by maintainers.
I also rearranged the checklist to have optional stuff at the bottom.
Checklist: