<br><br><div class="gmail_quote">2012/2/2 Stéfan van der Walt <span dir="ltr"><<a href="mailto:stefan@sun.ac.za">stefan@sun.ac.za</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hey Matthew<br>
<div class="im"><br>
On Wed, Feb 1, 2012 at 11:42 PM, Matthew Brett <<a href="mailto:matthew.brett@gmail.com">matthew.brett@gmail.com</a>> wrote:<br>
> I mean - are commit rights for:<br>
><br>
> * making your own commits OR<br>
> * merging other people's pull requests<br>
><br>
> I think I'm asking the same old git vs svn workflow question.<br>
<br>
</div>I think the "policy" for #1 is currently to make a PR, wait for<br>
feedback, and, if none is forthcoming, to proceed with the merge<br>
unless you know the change warrants further discussion (think of doc<br>
fixes, etc).  Essentially, it relies on the good judgement of the<br>
author.  Ideally, of course, authors shouldn't have to merge their own<br>
PRs, and Ralf and Warren have been very good at reviewing and merging.<br>
<span class="HOEnZb"></span></blockquote><div><br>Indeed, nothing (except truly trivial things like fixing a typo) should go in without review. Of course it's still possible to merge your own commits if another developer has reviewed them and commented that it's OK to do so. <br>
<br>Besides that, I think commit rights are also a kind of recognition for making significant good quality contributions, and that it's healthy for scipy to have a larger number of active developers which can review and merge PRs.<br>
<br>Cheers,<br>Ralf<br></div></div>