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 Thu, Sep 6, 2018 at 10:37 PM Brett Cannon <brett@python.org> wrote:
On Wed, 5 Sep 2018 at 13:37 Zachary Ware <zachary.ware+pydev@gmail.com> wrote:
What about no label at all on core dev PRs by default, but if one gets added the standard flow is followed?
I would be fine with that. What do others think?
+1
If no one beats me to it by the end of this month, I can submit a PR that implements Zachary's suggestion..
--Berker _______________________________________________ core-workflow mailing list -- core-workflow@python.org To unsubscribe send an email to core-workflow-leave@python.org https://mail.python.org/mm3/mailman3/lists/core-workflow.python.org/ This list is governed by the PSF Code of Conduct: https://www.python.org/psf/codeofconduct