Review Request 32323: DRY up PMD configuration.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32323/ --- Review request for Aurora, Joshua Cohen and Bill Farner. Repository: aurora Description --- DRY up PMD configuration. Diffs - build.gradle 4a13e5c4281de53d0ea991235502943a439d1310 config/pmd/custom.xml 7026d04d8e5c2f185a19c7c44d44698b2fb846c9 config/pmd/design.xml 2741e407e35826e73bd4dc8335c0ef6294086398 config/pmd/imports.xml 40dc2260a3c27fd66be352a5974427dcf165ab34 config/pmd/logging-java.xml 66fecbf14aa61f340678dc23d57a1c074cad824a config/pmd/naming.xml 233352d57857a0fe09ddbfe9228b48d4c09bb624 Diff: https://reviews.apache.org/r/32323/diff/ Testing --- ./gradlew -Pq pmdMain -d diffed the Using rule log lines from before and after: ``` % diff -u (sort -u old-rules.txt) (sort -u new-rules.txt) % ``` Thanks, Kevin Sweeney
Re: Review Request 32323: DRY up PMD configuration.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32323/#review77260 --- Ship it! Master (91aec8c) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing @ReviewBot retry - Aurora ReviewBot On March 20, 2015, 6:56 p.m., Kevin Sweeney wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32323/ --- (Updated March 20, 2015, 6:56 p.m.) Review request for Aurora, Joshua Cohen and Bill Farner. Repository: aurora Description --- DRY up PMD configuration. Diffs - build.gradle 4a13e5c4281de53d0ea991235502943a439d1310 config/pmd/custom.xml 7026d04d8e5c2f185a19c7c44d44698b2fb846c9 config/pmd/design.xml 2741e407e35826e73bd4dc8335c0ef6294086398 config/pmd/imports.xml 40dc2260a3c27fd66be352a5974427dcf165ab34 config/pmd/logging-java.xml 66fecbf14aa61f340678dc23d57a1c074cad824a config/pmd/naming.xml 233352d57857a0fe09ddbfe9228b48d4c09bb624 Diff: https://reviews.apache.org/r/32323/diff/ Testing --- ./gradlew -Pq pmdMain -d diffed the Using rule log lines from before and after: ``` % diff -u (sort -u old-rules.txt) (sort -u new-rules.txt) % ``` Thanks, Kevin Sweeney
Re: Review Request 32323: DRY up PMD configuration.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32323/#review77261 --- Ship it! Thanks! Good find! - Bill Farner On March 20, 2015, 6:56 p.m., Kevin Sweeney wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32323/ --- (Updated March 20, 2015, 6:56 p.m.) Review request for Aurora, Joshua Cohen and Bill Farner. Repository: aurora Description --- DRY up PMD configuration. Diffs - build.gradle 4a13e5c4281de53d0ea991235502943a439d1310 config/pmd/custom.xml 7026d04d8e5c2f185a19c7c44d44698b2fb846c9 config/pmd/design.xml 2741e407e35826e73bd4dc8335c0ef6294086398 config/pmd/imports.xml 40dc2260a3c27fd66be352a5974427dcf165ab34 config/pmd/logging-java.xml 66fecbf14aa61f340678dc23d57a1c074cad824a config/pmd/naming.xml 233352d57857a0fe09ddbfe9228b48d4c09bb624 Diff: https://reviews.apache.org/r/32323/diff/ Testing --- ./gradlew -Pq pmdMain -d diffed the Using rule log lines from before and after: ``` % diff -u (sort -u old-rules.txt) (sort -u new-rules.txt) % ``` Thanks, Kevin Sweeney