APEXCORE-305 - Enable checkstyle violations logging to console during maven build
Project: http://git-wip-us.apache.org/repos/asf/incubator-apex-core/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-apex-core/commit/59770d06 Tree: http://git-wip-us.apache.org/repos/asf/incubator-apex-core/tree/59770d06 Diff: http://git-wip-us.apache.org/repos/asf/incubator-apex-core/diff/59770d06 Branch: refs/heads/master Commit: 59770d0647bedbe81e794e3f3fd476e581c4339c Parents: 3b6c252 Author: Vlad Rozov <[email protected]> Authored: Mon Jan 11 15:04:46 2016 -0800 Committer: Vlad Rozov <[email protected]> Committed: Mon Jan 11 15:14:01 2016 -0800 ---------------------------------------------------------------------- api/pom.xml | 3 --- bufferserver/pom.xml | 4 +--- common/pom.xml | 3 --- engine/pom.xml | 5 +++++ pom.xml | 6 ------ 5 files changed, 6 insertions(+), 15 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-apex-core/blob/59770d06/api/pom.xml ---------------------------------------------------------------------- diff --git a/api/pom.xml b/api/pom.xml index 4c8f00b..d5ea932 100644 --- a/api/pom.xml +++ b/api/pom.xml @@ -70,9 +70,6 @@ <plugin> <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-checkstyle-plugin</artifactId> - <configuration> - <logViolationsToConsole>true</logViolationsToConsole> - </configuration> </plugin> </plugins> </build> http://git-wip-us.apache.org/repos/asf/incubator-apex-core/blob/59770d06/bufferserver/pom.xml ---------------------------------------------------------------------- diff --git a/bufferserver/pom.xml b/bufferserver/pom.xml index efc5b5e..aa79c0e 100644 --- a/bufferserver/pom.xml +++ b/bufferserver/pom.xml @@ -48,10 +48,8 @@ </plugin> --> <plugin> + <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-checkstyle-plugin</artifactId> - <configuration> - <consoleOutput>true</consoleOutput> - </configuration> </plugin> </plugins> </build> http://git-wip-us.apache.org/repos/asf/incubator-apex-core/blob/59770d06/common/pom.xml ---------------------------------------------------------------------- diff --git a/common/pom.xml b/common/pom.xml index 5c2c98f..a749cea 100644 --- a/common/pom.xml +++ b/common/pom.xml @@ -55,9 +55,6 @@ <plugin> <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-checkstyle-plugin</artifactId> - <configuration> - <logViolationsToConsole>true</logViolationsToConsole> - </configuration> </plugin> </plugins> </build> http://git-wip-us.apache.org/repos/asf/incubator-apex-core/blob/59770d06/engine/pom.xml ---------------------------------------------------------------------- diff --git a/engine/pom.xml b/engine/pom.xml index 3e43184..2b70aa6 100644 --- a/engine/pom.xml +++ b/engine/pom.xml @@ -33,6 +33,10 @@ <name>Apache Apex (incubating) Stream Processing Engine</name> + <properties> + <checkstyle.console>false</checkstyle.console> + </properties> + <build> <finalName>${project.artifactId}</finalName> <plugins> @@ -146,6 +150,7 @@ <artifactId>maven-checkstyle-plugin</artifactId> <configuration> <maxAllowedViolations>3224</maxAllowedViolations> + <logViolationsToConsole>${checkstyle.console}</logViolationsToConsole> </configuration> </plugin> <plugin> http://git-wip-us.apache.org/repos/asf/incubator-apex-core/blob/59770d06/pom.xml ---------------------------------------------------------------------- diff --git a/pom.xml b/pom.xml index 1972d37..164645c 100644 --- a/pom.xml +++ b/pom.xml @@ -64,7 +64,6 @@ <!-- do not change jetty version as later versions have problems with DefaultServlet --> <jetty.version>8.1.10.v20130312</jetty.version> <license.skip>true</license.skip> - <checkstyle.console>false</checkstyle.console> <postNoticeText>The initial developer of the original code is
DataTorrent, Inc. (http://www.datatorrent.com)
Copyright (c) 2012 - 2015. All Rights Reserved.</postNoticeText> </properties> @@ -316,16 +315,11 @@ <goals> <goal>check</goal> </goals> - <configuration> - <failOnViolation>true</failOnViolation> - <logViolationsToConsole>${checkstyle.console}</logViolationsToConsole> - </configuration> </execution> </executions> <configuration> <configLocation>apex_checks.xml</configLocation> <suppressionsLocation>checkstyle-suppressions.xml</suppressionsLocation> - <suppressionsFileExpression>checkstyle.suppressions.file</suppressionsFileExpression> <includeTestSourceDirectory>true</includeTestSourceDirectory> </configuration> </plugin>
