You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ah...@apache.org on 2019/11/09 08:41:24 UTC

[commons-numbers] branch master updated: checkstyle:check and pmd:check to log violations to console.

This is an automated email from the ASF dual-hosted git repository.

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-numbers.git


The following commit(s) were added to refs/heads/master by this push:
     new bd83bb0  checkstyle:check and pmd:check to log violations to console.
bd83bb0 is described below

commit bd83bb05359fce6c1e427e452884687ab5f7a0ee
Author: Alex Herbert <ah...@apache.org>
AuthorDate: Sat Nov 9 08:41:20 2019 +0000

    checkstyle:check and pmd:check to log violations to console.
    
    Helps in detecting reasons for failure.
---
 pom.xml | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 115f5b0..8296f3b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -253,7 +253,10 @@
           <headerLocation>${numbers.parent.dir}/src/main/resources/checkstyle/license-header.txt</headerLocation>
           <suppressionsLocation>${numbers.parent.dir}/src/main/resources/checkstyle/checkstyle-suppressions.xml</suppressionsLocation>
           <enableRulesSummary>false</enableRulesSummary>
-          <logViolationsToConsole>false</logViolationsToConsole>
+          <!-- Output issues to console. -->
+          <consoleOutput>false</consoleOutput>
+          <!-- Output the detected violations to the console (for checkstyle:check). -->
+          <logViolationsToConsole>true</logViolationsToConsole>
           <failOnViolation>true</failOnViolation>
           <resourceExcludes>NOTICE.txt,LICENSE.txt,**/maven-archiver/pom.properties,**/resolver-status.properties</resourceExcludes>
           <excludes>**/generated/**.java</excludes>
@@ -294,6 +297,7 @@
           </dependency>
         </dependencies>
         <configuration>
+          <printFailingErrors>true</printFailingErrors>
           <!-- TODO: remove this when PMD has been fixed. -->
           <failOnViolation>false</failOnViolation>
           <targetJdk>${maven.compiler.target}</targetJdk>
@@ -436,6 +440,8 @@
         <artifactId>maven-pmd-plugin</artifactId>
         <version>${numbers.pmd.version}</version>
         <configuration>
+          <verbose>false</verbose>
+          <printFailingErrors>true</printFailingErrors>
           <targetJdk>${maven.compiler.target}</targetJdk>
           <skipEmptyReport>false</skipEmptyReport>
           <analysisCache>true</analysisCache>