> On Oct. 13, 2014, 4:23 p.m., Joshua Cohen wrote: > > buildSrc/src/main/groovy/org/apache/aurora/CoverageReportCheck.groovy, line > > 53 > > <https://reviews.apache.org/r/26662/diff/2/?file=719791#file719791line53> > > > > I'm not super familiar w/ groovy, any reason why we can't just inline > > the computeCoverage(...) call here (and below)? > > Bill Farner wrote: > This was a nuance in the way the assertion error would show - with > inlining, you wouldn't see the actual value for some reason. So you would > see "0.9" instead of "0.89 > 0.9". > > Bill Farner wrote: > I've gone ahead and inlined now, though, since in retrospect i don't > think we really care about the absolute value; and it's easy enough to get > from the coverage report HTML.
Technically you could write it in Java :) - Kevin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26662/#review56466 ----------------------------------------------------------- On Oct. 13, 2014, 4:36 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26662/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2014, 4:36 p.m.) > > > Review request for Aurora, Joshua Cohen and Kevin Sweeney. > > > Bugs: AURORA-833 > https://issues.apache.org/jira/browse/AURORA-833 > > > Repository: aurora > > > Description > ------- > > Making good on a TODO from https://reviews.apache.org/r/26574/ > > > Diffs > ----- > > .gitignore ad4206d179b46f1269fd58c62de9e6e62f4b8738 > build.gradle c200746ff9ce2d7ff973c22764d9413bf64636e7 > buildSrc/src/main/groovy/org/apache/aurora/CoverageReportCheck.groovy > PRE-CREATION > config/legacy_untested_classes.txt PRE-CREATION > > Diff: https://reviews.apache.org/r/26662/diff/ > > > Testing > ------- > > ./gradlew build -Pq > > > Thanks, > > Bill Farner > >