[IPython-dev] Policy for closing pull requests

Aaron Meurer asmeurer at gmail.com
Sat Aug 11 22:17:24 EDT 2012


One other important rule that we have in SymPy is that if we close a
pull request that still needs work (as opposed to an outright
rejection), we make sure that it is mentioned in an open issue.
Otherwise, it will be forgotten forever.

On Sat, Aug 11, 2012 at 7:46 PM, Brian Granger <ellisonbg at gmail.com> wrote:
> Hi,
>
> Ondrej has been going through the sympy pull request queue (they have
> 53 open, but had more than 70) and trying to close ones that are no
> longer active.  This as inspired me to think about these issues for
> IPython.  I am wondering if we can come up with a policy for closing
> pull requests.  Here is what I am thinking.
>
> * Let's use use pull requests for code that is actively being worked
> on and reviewed and that has a strong chance of being merged soon.
> * Open PRs should be in one of two states: waiting for review or
> waiting for additional code.  It should be obvious who the "person of
> next action" is.
> * When a PR is not in one of these states, it should be closed.
> * When a PR is in one of these states, but sits untouched for a long
> period of time, we close it and indicate in a comment what would need
> to be done to reopen it.
>
> In some rare cases we will outright reject a PR.  But in many cases,
> we will close PRs with a fairly positive statement like "this is
> promising, please reopen this PR after you do ..."  This is similar to
> the to "Someday/Maybe" category of Getting Things Done.

You could also use actual labels in the issue tracker to do this.

Aaron Meurer

>
> I think this would help us keep our PR/review workflow moving and
> encourage people to revisit PRs that are inactive.
>
> Thoughts?
>
> Cheers,
>
> Brian
>
>
> --
> Brian E. Granger
> Cal Poly State University, San Luis Obispo
> bgranger at calpoly.edu and ellisonbg at gmail.com
> _______________________________________________
> IPython-dev mailing list
> IPython-dev at scipy.org
> http://mail.scipy.org/mailman/listinfo/ipython-dev



More information about the IPython-dev mailing list