Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40129/#review106309 --- Bad patch! Reviews applied: [40225, 40241, 39410] Failed command:

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-12 Thread Artem Harutyunyan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40129/ --- (Updated Nov. 12, 2015, 10:04 a.m.) Review request for mesos, Joris Van Remoort

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-12 Thread Artem Harutyunyan
> On Nov. 10, 2015, 12:23 p.m., Vinod Kone wrote: > > support/apply-review.sh, line 5 > > > > > > do we even need this script anymore? is this for backwards compatiblity > > with existing tooling? is that plan to kill

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40129/#review106228 --- Patch looks great! Reviews applied: [40225, 39410, 39420, 40129]

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40129/#review106225 --- Patch looks great! Reviews applied: [40225, 39410, 39420, 40129]

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-12 Thread Artem Harutyunyan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40129/ --- (Updated Nov. 12, 2015, 2:54 a.m.) Review request for mesos, Joris Van Remoorte

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-11 Thread Vinod Kone
> On Nov. 10, 2015, 8:23 p.m., Vinod Kone wrote: > > support/apply-review.sh, line 5 > > > > > > do we even need this script anymore? is this for backwards compatiblity > > with existing tooling? is that plan to kill

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-11 Thread Artem Harutyunyan
> On Nov. 10, 2015, 12:23 p.m., Vinod Kone wrote: > > support/apply-review.sh, line 5 > > > > > > do we even need this script anymore? is this for backwards compatiblity > > with existing tooling? is that plan to kill

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-10 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40129/#review105943 --- Ship it! support/apply-review.sh

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-10 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40129/#review105811 --- Patch looks great! Reviews applied: [38705, 38883, 39410, 39420, 4

Re: Review Request 40129: Updated apply-review.sh to use apply-reviews.py.

2015-11-10 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40129/#review105808 --- Bad patch! Reviews applied: [38705, 38883, 39410] Failed command: