"Awaiting merge" label for PRs from core devs

Hi. When core dev creates PR, "Awaiting merge" label is set automatically. It makes many "awaiting merge" PRs. It makes the label less useful, in my opinion. How about stop setting the label automatically? If the author wants review from other core-devs, they can set "awaiting core review" label manually. Other option is set "awaiting review" label instead of "awaiting merge", like PRs from other contributors. Regards, -- INADA Naoki <songofacandy@gmail.com>

On Wed, Sep 5, 2018 at 3:28 PM INADA Naoki <songofacandy@gmail.com> wrote:
Other option is set "awaiting review" label instead of "awaiting merge", like PRs from other contributors.
+1. Personally, most of the time I request a review from another core developer, so setting the "awaiting review" label seems reasonable to me. --Berker

So the reasoning behind setting "awaiting merge" is because the "needs" label is meant to represent what is holding up the PR from being closed, and so a PR from a core dev is really just blocked on merging since they don't have to receive a review. Now, if people would rather move it over to "needs review" by default like any other PR then that's fine as well, but I don't know if that's going to help with INADA-san's worry about so many issues making a label less useful. On Wed, 5 Sep 2018 at 05:45 Berker Peksağ <berker.peksag@gmail.com> wrote:

On Wed, Sep 5, 2018 at 3:31 PM Brett Cannon <brett@python.org> wrote:
So the reasoning behind setting "awaiting merge" is because the "needs" label is meant to represent what is holding up the PR from being closed, and so a PR from a core dev is really just blocked on merging since they don't have to receive a review.
Now, if people would rather move it over to "needs review" by default like any other PR then that's fine as well, but I don't know if that's going to help with INADA-san's worry about so many issues making a label less useful.
What about no label at all on core dev PRs by default, but if one gets added the standard flow is followed? -- Zach

I prefer keeping the labeling as is, so core devs PR get "awaiting merge" label upon creation. If a core dev actually want someone else to review it, I suggest using the "request review" feature, and select another core dev to review the PR. At that time, bedevere should apply the "awaiting core review" label. At least, that would be my expectation. If it doesn't do that, then that's the change we should implement, instead of the "no label by default". Mariatta ᐧ On Thu, Sep 6, 2018 at 12:47 PM Berker Peksağ <berker.peksag@gmail.com> wrote:

On Tue, 11 Sep 2018 at 05:13, Brett Cannon <brett@python.org> wrote:
If I'm specifically waiting for feedback on a PR, I'll generally put an explicit "WIP: " (for work-in-progress) in the title to warn other folks not to merge it yet (it's a habit picked up when using GitLab, as that will actually prevent merging if you do that). So if folks don't want to make "No label" the default, then the convention could instead be to add "WIP: " to tell Bedevere to skip adding the "Awaiting merge" label. Cheers, Nick. -- Nick Coghlan | ncoghlan@gmail.com | Brisbane, Australia

On Tue, 11 Sep 2018 at 07:33 Nick Coghlan <ncoghlan@gmail.com> wrote:
We could set up a separate bot at this point, but with the new automerge bot this won't be such an issue as people will simply not add the "automerge" label until they are ready.
I think we may want to hold off on any changes until we see what happens to our standard practices with the automerge bot. -Brett

FYI I opened a poll at https://discuss.python.org/t/poll-what-label-should-prs-from-core-devs-autom... to settle this question of what label core dev PRs whould automatically receive. On Tue, Sep 11, 2018 at 2:49 PM Brett Cannon <brett@python.org> wrote:

On Wed, Sep 5, 2018 at 3:28 PM INADA Naoki <songofacandy@gmail.com> wrote:
Other option is set "awaiting review" label instead of "awaiting merge", like PRs from other contributors.
+1. Personally, most of the time I request a review from another core developer, so setting the "awaiting review" label seems reasonable to me. --Berker

So the reasoning behind setting "awaiting merge" is because the "needs" label is meant to represent what is holding up the PR from being closed, and so a PR from a core dev is really just blocked on merging since they don't have to receive a review. Now, if people would rather move it over to "needs review" by default like any other PR then that's fine as well, but I don't know if that's going to help with INADA-san's worry about so many issues making a label less useful. On Wed, 5 Sep 2018 at 05:45 Berker Peksağ <berker.peksag@gmail.com> wrote:

On Wed, Sep 5, 2018 at 3:31 PM Brett Cannon <brett@python.org> wrote:
So the reasoning behind setting "awaiting merge" is because the "needs" label is meant to represent what is holding up the PR from being closed, and so a PR from a core dev is really just blocked on merging since they don't have to receive a review.
Now, if people would rather move it over to "needs review" by default like any other PR then that's fine as well, but I don't know if that's going to help with INADA-san's worry about so many issues making a label less useful.
What about no label at all on core dev PRs by default, but if one gets added the standard flow is followed? -- Zach

I prefer keeping the labeling as is, so core devs PR get "awaiting merge" label upon creation. If a core dev actually want someone else to review it, I suggest using the "request review" feature, and select another core dev to review the PR. At that time, bedevere should apply the "awaiting core review" label. At least, that would be my expectation. If it doesn't do that, then that's the change we should implement, instead of the "no label by default". Mariatta ᐧ On Thu, Sep 6, 2018 at 12:47 PM Berker Peksağ <berker.peksag@gmail.com> wrote:

On Tue, 11 Sep 2018 at 05:13, Brett Cannon <brett@python.org> wrote:
If I'm specifically waiting for feedback on a PR, I'll generally put an explicit "WIP: " (for work-in-progress) in the title to warn other folks not to merge it yet (it's a habit picked up when using GitLab, as that will actually prevent merging if you do that). So if folks don't want to make "No label" the default, then the convention could instead be to add "WIP: " to tell Bedevere to skip adding the "Awaiting merge" label. Cheers, Nick. -- Nick Coghlan | ncoghlan@gmail.com | Brisbane, Australia

On Tue, 11 Sep 2018 at 07:33 Nick Coghlan <ncoghlan@gmail.com> wrote:
We could set up a separate bot at this point, but with the new automerge bot this won't be such an issue as people will simply not add the "automerge" label until they are ready.
I think we may want to hold off on any changes until we see what happens to our standard practices with the automerge bot. -Brett

FYI I opened a poll at https://discuss.python.org/t/poll-what-label-should-prs-from-core-devs-autom... to settle this question of what label core dev PRs whould automatically receive. On Tue, Sep 11, 2018 at 2:49 PM Brett Cannon <brett@python.org> wrote:
participants (6)
-
Berker Peksağ
-
Brett Cannon
-
INADA Naoki
-
Mariatta Wijaya
-
Nick Coghlan
-
Zachary Ware