-----------------------------------------------------------
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
> 
>

Reply via email to