Numpy pull requests getting out of hand.
![](https://secure.gravatar.com/avatar/96dd777e397ab128fedab46af97a3a4a.jpg?s=120&d=mm&r=g)
Hi All, There are now 130 open numpy pull requests and it seems almost impossible to keep that number down. My personal decision is that I am going to ignore any new enhancements for the next couple of months and only merge bug fixes, tests, house keeping (style, docs, deprecations), and old PRs. I would also request that other maintainers start looking a taking care of older PRs, either cleaning them up and merging, or closing them. Chuck
![](https://secure.gravatar.com/avatar/c0da24f75f763b6bac90b519064f30b3.jpg?s=120&d=mm&r=g)
I don't like that approach, closing PRs with valuable code causes them to get lost in the much larger set of closed ones. Instead we could tag them appropriately so they can be found by parties interested in using or finishing them. These could also be used for new contributors to work on. You could also tag and close them, but I find it makes them harder to discover especially for outsiders who are not aware of this policy. Also we could extend our contribution guidelines to note that reviewing existing PRs can be a much more valuable contribution than adding new code. Possibly adding some review guidelines. On 31.01.2016 20:25, Jeff Reback wrote:
![](https://secure.gravatar.com/avatar/bd4477dc26bf9941268fbfa05abdeae6.jpg?s=120&d=mm&r=g)
On Mon, Feb 1, 2016 at 11:22 PM, Julian Taylor <jtaylor.debian@googlemail.com> wrote:
FWIW, I second this. http://scipy.github.io/devdocs/hacking.html mentions "Contributing new code" and "Contributing by helping maintain existing code". We should add "Contributing by reviewing open PRs" and "Contributing by rebasing stalled PRs". It's very much not obvious to would-be-contributors that either is welcome or that the latter is actually possible. I don't even think we need to detail guidelines too much, apart from stressing that we want to preserve the original commit authorship. And maybe "briefly introduce yourself if we don't know you yet". W.r.t. tagging, over at scipy we have a "needs-work" tag. As of today, there is also the "incomplete" tag, which I do not think conveys the message --- we should stress the call for action, not a limbo status. At a minimum, we could adopt a "needs-review" tag from e.g. matplotlib and introduce something like "up-for-grabs" or "needs-champion" tag. [Of course, the "we should" above is more like, "I think it might make sense for scipy, maybe also numpy could consider this". ] My 2 kopeiki, Evgeni
![](https://secure.gravatar.com/avatar/c0da24f75f763b6bac90b519064f30b3.jpg?s=120&d=mm&r=g)
I don't like that approach, closing PRs with valuable code causes them to get lost in the much larger set of closed ones. Instead we could tag them appropriately so they can be found by parties interested in using or finishing them. These could also be used for new contributors to work on. You could also tag and close them, but I find it makes them harder to discover especially for outsiders who are not aware of this policy. Also we could extend our contribution guidelines to note that reviewing existing PRs can be a much more valuable contribution than adding new code. Possibly adding some review guidelines. On 31.01.2016 20:25, Jeff Reback wrote:
![](https://secure.gravatar.com/avatar/bd4477dc26bf9941268fbfa05abdeae6.jpg?s=120&d=mm&r=g)
On Mon, Feb 1, 2016 at 11:22 PM, Julian Taylor <jtaylor.debian@googlemail.com> wrote:
FWIW, I second this. http://scipy.github.io/devdocs/hacking.html mentions "Contributing new code" and "Contributing by helping maintain existing code". We should add "Contributing by reviewing open PRs" and "Contributing by rebasing stalled PRs". It's very much not obvious to would-be-contributors that either is welcome or that the latter is actually possible. I don't even think we need to detail guidelines too much, apart from stressing that we want to preserve the original commit authorship. And maybe "briefly introduce yourself if we don't know you yet". W.r.t. tagging, over at scipy we have a "needs-work" tag. As of today, there is also the "incomplete" tag, which I do not think conveys the message --- we should stress the call for action, not a limbo status. At a minimum, we could adopt a "needs-review" tag from e.g. matplotlib and introduce something like "up-for-grabs" or "needs-champion" tag. [Of course, the "we should" above is more like, "I think it might make sense for scipy, maybe also numpy could consider this". ] My 2 kopeiki, Evgeni
participants (4)
-
Charles R Harris
-
Evgeni Burovski
-
Jeff Reback
-
Julian Taylor