Cool, thanks. I've re-enabled the precommit job, reconfigured it, and it seems to be working again. LMK if you notice any issues.
Patrick On Mon, Sep 12, 2016 at 8:22 PM, Raúl Gutiérrez Segalés <r...@apache.org> wrote: > On 12 September 2016 at 17:48, Patrick Hunt <ph...@apache.org> wrote: > >> On Mon, Sep 12, 2016 at 10:15 AM, Raúl Gutiérrez Segalés >> <r...@itevenworks.net> wrote: >> > On 12 September 2016 at 09:58, Patrick Hunt <ph...@apache.org> wrote: >> > >> >> Here it is, please take a look, review, and commit it to master >> >> (remember, needs to be git now :-) ) >> >> https://issues.apache.org/jira/browse/ZOOKEEPER-2576 >> > >> > >> > Merged: >> > https://git-wip-us.apache.org/repos/asf?p=zookeeper.git;a=commitdiff;h= >> 8c4082647f89b0a92fa00a2af8de84b3c7314e23 >> > >> > This is only needed in master? >> > >> >> Only on master. Our current pre-commit is only for master. If we move >> to something like Yetus I believe they will also check branches. >> >> Raul/Ben/et.al. can you commit the second part of the patch? I >> attached it to ZOOKEEPER-2576. Bit of a cleanup on the command naming >> (missed build.xml changes). Thanks. >> > > Merged: > https://git-wip-us.apache.org/repos/asf?p=zookeeper.git;a=commitdiff;h=b2a484cfe743116d2531fe5d1e1d78b3960c511e > > > -rgs
