----------------------------------------------------------- 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 <https://reviews.apache.org/r/27770/#comment101998> diffs and latest_diff_time both seem unused here? build-support/jenkins/review_feedback.py <https://reviews.apache.org/r/27770/#comment102005> Is it worth excluding src/main/resources? (I suspect the chances of a resource-only change are pretty slim, so might not be worth the effort for the rare false positive). - Joshua Cohen On Nov. 8, 2014, 4:24 a.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27770/ > ----------------------------------------------------------- > > (Updated Nov. 8, 2014, 4:24 a.m.) > > > Review request for Aurora, David McLaughlin and Joshua Cohen. > > > Repository: aurora > > > Description > ------- > > Make it easier to request another ReviewBot run, and flag diffs that seem to > lack test coverage. > > > Diffs > ----- > > build-support/jenkins/review_feedback.py > 718c538631a7977589e13ddae8db9301f3887c7e > > Diff: https://reviews.apache.org/r/27770/diff/ > > > Testing > ------- > > Ran locally to trigger a build here: https://reviews.apache.org/r/27710/ > > > Thanks, > > Bill Farner > >