[Numpy-discussion] commit rights for Nathaniel

[Charles R Harris]

On Tue, Jun 5, 2012 at 11:52 AM, Charles R Harris <charlesr.harris at gmail.com
> wrote:

>
>
> On Tue, Jun 5, 2012 at 10:25 AM, Nathaniel Smith <njs at pobox.com> wrote:
>
>> On Tue, Jun 5, 2012 at 4:19 PM, Charles R Harris
>> <charlesr.harris at gmail.com> wrote:
>> >
>> >
>> > On Sun, Jun 3, 2012 at 12:04 PM, Ralf Gommers <
>> ralf.gommers at googlemail.com>
>> > wrote:
>> >>
>> >>
>> >>
>> >> On Sun, Jun 3, 2012 at 6:43 PM, Charles R Harris
>> >> <charlesr.harris at gmail.com> wrote:
>> >>>
>> >>> Hi All,
>> >>>
>> >>> Numpy is approaching a time of transition. Ralf will be concentrating
>> his
>> >>> efforts on Scipy
>> >>
>> >>
>> >> I'll write a separate post on that asap.
>> >>
>> >>>
>> >>> and I will be cutting back on my work on Numpy.
>> >>
>> >>
>> >> I sincerely hope you don't cut back on your work too much Charles. You
>> >> have done an excellent job as "chief maintainer" over the last years.
>> >>
>> >>> The 1.7 release looks to be delayed and I suspect that the Continuum
>> >>> Analytics folks will become increasingly dedicated to the big data
>> push. We
>> >>> need new people to carry things forward and I think Nathaniel can
>> pick up
>> >>> part of the load.
>> >>
>> >>
>> >> Assuming he wants them, I am definitely +1 on giving Nathaniel commit
>> >> rights. His recent patches and debugging of issues were of high
>> quality and
>> >> very helpful.
>> >>
>> >
>> > OK, I went ahead and added him whether he wants it or not ;)
>>
>> Hah. Thanks!
>>
>> Is there a "committers guide" anywhere? By default I would assume that
>> the rules are pretty much -- continue sending pull requests for my own
>> changes (unless a trivial typo fix in a comment or something), go
>> ahead and merge anyone else's pull request where things seem okay and
>> my best judgement is we have consensus, fix things if my judgement was
>> wrong? But I don't want to step on any toes...
>>
>>
> You can commit your own stuff also if someone signs off on it or it seems
> uncontroversial and has sat there for a while. It's mostly a judgement call.
>
> For the commits themselves, the github button doesn't do fast forward or
> whitespace cleanup, so I have the following alias in .git/config
>
> getpatch = !sh -c 'git co -b pull-$1 master &&\
>            curl https://github.com/numpy/nump/pull/$1.patch|\
>            git am -3 --whitespace=strip' -
>
> which opens a new branch pull-nnn and is useful for the bigger commits so
> they can be tested and then merged with master before pushing. The
> non-trivial commits should be tested with at least Python 2.4, 2.7, and
> 3.2. I also suggest running the one-file build for changes in core since
> most developers do the separate file thing and sometimes fail to catch
> single file build problems.
>
> Keep an eye on coding style, otherwise it will drift.
>
>
And keep in mind that part of your job is to train new committers and help
bring them up to speed. See yourself as a recruiter as well as a reviewer.

Chuck
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/numpy-discussion/attachments/20120605/48903f7e/attachment.html>