Re: Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage.

2014-11-10 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27770/ --- (Updated Nov. 11, 2014, 3:50 a.m.) Review request for Aurora, David McLaughlin

Re: Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage.

2014-11-10 Thread Bill Farner
> On Nov. 10, 2014, 7:55 p.m., Kevin Sweeney wrote: > > build-support/jenkins/review_feedback.py, line 84 > > > > > > consider making this case insensitive Done. - Bill -

Re: Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage.

2014-11-10 Thread Bill Farner
> On Nov. 10, 2014, 6:47 p.m., Joshua Cohen wrote: > > build-support/jenkins/review_feedback.py, lines 75-76 > > > > > > diffs and latest_diff_time both seem unused here? Thanks, fixed. > On Nov. 10, 2014, 6:47 p.m.,

Re: Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage.

2014-11-10 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27770/#review60638 --- Ship it! build-support/jenkins/review_feedback.py

Re: Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage.

2014-11-10 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27770/#review60650 --- Ship it! Awesome. - David McLaughlin On Nov. 8, 2014, 4:24 a.m.,

Re: Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage.

2014-11-10 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27770/#review60631 --- Ship it! build-support/jenkins/review_feedback.py

Re: Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage.

2014-11-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27770/#review60472 --- Ship it! Master (54b0262) is green with this patch. ./build-suppo

Review Request 27770: Make it easier to request another ReviewBot run, and flag diffs that seem to lack test coverage.

2014-11-07 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27770/ --- Review request for Aurora, David McLaughlin and Joshua Cohen. Repository: auror