Re: Review Request 62793: Added cpplint to the list of excluded files by Python linter.

2017-10-11 Thread Armand Grillet
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62793/ --- (Updated Oct. 11, 2017, 9:28 a.m.) Review request for mesos, Alexander

Re: Review Request 62793: Added cpplint to the list of excluded files by Python linter.

2017-10-10 Thread Armand Grillet
> On Oct. 10, 2017, 2:29 p.m., Benjamin Bannier wrote: > > support/mesos-style.py > > Lines 346 (patched) > > > > > > Any reason we are not more explicit here, e.g., `r'cpplint.py'`? The previous lines were only

Re: Review Request 62793: Added cpplint to the list of excluded files by Python linter.

2017-10-10 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62793/#review187534 --- Fix it, then Ship it! support/mesos-style.py Lines 346

Re: Review Request 62793: Added cpplint to the list of excluded files by Python linter.

2017-10-10 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62793/#review187517 --- FAIL: Some Mesos tests failed. Reviews applied: `['62793']`

Re: Review Request 62793: Added cpplint to the list of excluded files by Python linter.

2017-10-10 Thread Armand Grillet
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62793/ --- (Updated Oct. 10, 2017, 9:57 a.m.) Review request for mesos, Alexander