Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-15 Thread Avinash sridharan
> On Jan. 14, 2016, 8:46 p.m., Michael Park wrote: > > Please update the `Summary` to be < 72 chars to make Reviewbot happy. Could > > you also submit a patch for the places that violate the rule we're > > enforcing here and make this patch depend on it? Again, so that this way > > Reviewbot

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-15 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/ --- (Updated Jan. 15, 2016, 5:41 p.m.) Review request for mesos and Michael Park.

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-15 Thread Michael Park
> On Jan. 14, 2016, 8:46 p.m., Michael Park wrote: > > Please update the `Summary` to be < 72 chars to make Reviewbot happy. Could > > you also submit a patch for the places that violate the rule we're > > enforcing here and make this patch depend on it? Again, so that this way > > Reviewbot

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/#review114609 --- Bad patch! Reviews applied: [42325, 42326, 41617] Failed

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-14 Thread Avinash sridharan
> On Jan. 14, 2016, 8:46 p.m., Michael Park wrote: > > Please update the `Summary` to be < 72 chars to make Reviewbot happy. Could > > you also submit a patch for the places that violate the rule we're > > enforcing here and make this patch depend on it? Again, so that this way > > Reviewbot

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-14 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/ --- (Updated Jan. 15, 2016, 3:35 a.m.) Review request for mesos and Michael Park.

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-14 Thread Avinash sridharan
> On Jan. 14, 2016, 11:02 p.m., Michael Park wrote: > > support/cpplint.py, line 2641 > > > > > > Sorry for the follow-up review after a `ShipIt`, but it looks like we > > missed something here. The comment says:

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-14 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/ --- (Updated Jan. 14, 2016, 10:51 p.m.) Review request for mesos and Michael Park.

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-14 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/#review114602 --- Ship it! support/cpplint.py (lines 32 - 33)

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/#review114662 --- Bad patch! Reviews applied: [42325, 42326, 41617] Failed

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-14 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/#review114563 --- Ship it! Please update the `Summary` to be < 72 chars to make

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-13 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/ --- (Updated Jan. 13, 2016, 8:15 p.m.) Review request for mesos and Michael Park.

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/#review114417 --- Bad patch! Reviews applied: [41617] Failed command:

Re: Review Request 41617: Added a new category called whitespace/mesos-comments to capture missing, leading, white-space in comments

2016-01-13 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41617/ --- (Updated Jan. 14, 2016, 1:59 a.m.) Review request for mesos and Michael Park.