<p dir="ltr"><br>
On 14 Apr 2014 01:56, "Stephen J. Turnbull" <<a href="mailto:stephen@xemacs.org">stephen@xemacs.org</a>> wrote:<br>
><br>
> <a href="mailto:martin@v.loewis.de">martin@v.loewis.de</a> writes:<br>
><br>
>  > For gaining commit access, it's really more important that the patch<br>
>  > is factually finished, than that it's author believes it to. If people<br>
>  > get it right the first time often enough, they get commit access.<br>
><br>
> Yes, that's what I had in mind, but I guess I explained it poorly.</p>
<p dir="ltr">We should capture this discussion clearly in the dev guide. Even if we switch to a core reviewer model at some point (as I propose in PEP 462), the criteria for core reviewer status will match those for core commiter status.</p>

<p dir="ltr">There are actually a few things I'm personally looking for:</p>
<p dir="ltr">* good judgement on when a patch is "finished enough" to merge<br>
* good judgement on whether a change is a new feature or a bug fix<br>
* good judgement whether a new feature is worth the additional cognitive burden<br>
* good ability to assess backwards compatibility risks<br>
* sufficient humility to answer "I don't know" to the above questions when appropriate and ask the relevant domain experts, their sponsoring mentor, the core-mentorship list or python-dev at large for advice on what to do</p>

<p dir="ltr">It's that last one which is really most critical - even Guido asks for additional input when he's uncertain about something, and that's a key part of why I trust his decisions on those rare occasions when he finds it necessary to exercise BDFL fiat (although his long history of demonstrating excellent language design instincts certainly helps!)</p>

<p dir="ltr">Cheers,<br>
Nick.</p>
<p dir="ltr">><br>
><br>
> _______________________________________________<br>
> Python-Dev mailing list<br>
> <a href="mailto:Python-Dev@python.org">Python-Dev@python.org</a><br>
> <a href="https://mail.python.org/mailman/listinfo/python-dev">https://mail.python.org/mailman/listinfo/python-dev</a><br>
> Unsubscribe: <a href="https://mail.python.org/mailman/options/python-dev/ncoghlan%40gmail.com">https://mail.python.org/mailman/options/python-dev/ncoghlan%40gmail.com</a><br>
</p>