<div dir="ltr"><div>Hello,</div><div><br></div><div>Overall, I think this is a good idea, but I would like specifics on what "uncontroversial but uninteresting" PR mean. IMO, that should exclude any PR with API changes.</div><div><br></div><div>Cheers,</div><div>N<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 15 Jan 2019 at 06:00, Antony Lee <<a href="mailto:anntzer.lee@gmail.com">anntzer.lee@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div>Hi all,</div><div><br></div><div>During the weekly dev call, I proposed to amend the PR merge rules, following an initial comment on Github [<a href="https://github.com/matplotlib/matplotlib/pull/13173#issuecomment-453921220" target="_blank">https://github.com/matplotlib/matplotlib/pull/13173#issuecomment-453921220</a>].  There was general agreement among the devs present (Tom, Hannah, Jody, and myself), so I'm putting it here for discussion.  The objective of this change is to prevent "uncontroversial but uninteresting" PRs from falling into oblivion, and try to decrease the size of the open PR stack.</div><div><br></div><div>The current rule is that a PR needs positive reviews (from two committers, and excluding the author if a committer) to be merged, except that doc-only PRs (docstrings, rst) only needs a single positive review.</div><div><br></div><div>I am proposing that, if a (non-doc) PR already has one positive review, but no activity on that PR has occurred for two weeks (exact time interval up to bikeshedding) [Jody suggests: and the PR has 100% code coverage], then a committer (either the first reviewer, or the author if a committer) can suggest that it be merged on the basis of that single review.  To do so, the "sponsor" should ping all developers (@matplotlib/developers) on that issue indicating that intent, and add a "single-review-merge" label on the PR (so that these PRs can easily be found).  Committers are encouraged to review the PR to accept and merge it or reject it or request changes on it; but they can also just indicate that they consider the PR sufficiently complex that a proper second review is needed before merging, or request an extension, etc.  To do so they should still leave a "reject" review, even if just saying "objecting to single-review merged; anyone can dismiss after a second review".  However, if within another two weeks, no committer voiced any objection (explicitly, i.e. by rejecting), then the PR can indeed be merged on the basis of that single review.</div><div><br></div><div>To avoid overwhelming the system, any committer can only "sponsor" a single PR at a time.</div><div><br></div><div>Thoughts?</div><div><br></div><div>Antony</div></div></div>
_______________________________________________<br>
Matplotlib-devel mailing list<br>
<a href="mailto:Matplotlib-devel@python.org" target="_blank">Matplotlib-devel@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/matplotlib-devel" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/matplotlib-devel</a><br>
</blockquote></div>