-
Notifications
You must be signed in to change notification settings - Fork 97
Exclude drafts from auto-backport nomination #2170
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
Exclude drafts from auto-backport nomination #2170
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.
Thanks, this makes sense, though I'm not super familiar with triagebot's codebase so rolling a triagebot reviewer :3
|
r? triagebot |
31cc5f1 to
68de8c8
Compare
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'm not sure what should happen when the PR is undrafted - should this handler run to check for backports? If yes, then we might want to add IssuesAction::ReadyForReview, because I don't think that Edited runs when the PR is converted away from a draft.
68de8c8 to
2df24a0
Compare
2df24a0 to
d4cd168
Compare
|
@Kobzol back to this old small patch (had forgot about yr comment). I inverted the check boolean logic and added your suggestions. Feel to have a look when you have a sec. thanks |
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.
Looks good, thanks!
We have a mechanism (see 2556583) that auto-nominates pull requests for a backport if they fix a P-high/P-critical regression.
@jieyouxu suggested in this Zulip comment to exclude pull requests drafts from this mechanism (which makes sense), so this patch implements this suggestion.
r? @jieyouxu
thanks for a review :)