You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by se...@apache.org on 2015/07/09 07:34:27 UTC

directory-kerby git commit: DIRKRB-326: Activate checkstyle check in normal build, and fail on violation

Repository: directory-kerby
Updated Branches:
  refs/heads/master 1d312c9f6 -> c1673f7a4


DIRKRB-326: Activate checkstyle check in normal build, and fail on violation


Project: http://git-wip-us.apache.org/repos/asf/directory-kerby/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-kerby/commit/c1673f7a
Tree: http://git-wip-us.apache.org/repos/asf/directory-kerby/tree/c1673f7a
Diff: http://git-wip-us.apache.org/repos/asf/directory-kerby/diff/c1673f7a

Branch: refs/heads/master
Commit: c1673f7a4d5f4a797c0ead96231c67ea1b91cd64
Parents: 1d312c9
Author: Stefan Seelmann <ma...@stefan-seelmann.de>
Authored: Thu Jul 9 07:33:21 2015 +0200
Committer: Stefan Seelmann <ma...@stefan-seelmann.de>
Committed: Thu Jul 9 07:33:21 2015 +0200

----------------------------------------------------------------------
 pom.xml | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/directory-kerby/blob/c1673f7a/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index f292cc3..a6a3327 100644
--- a/pom.xml
+++ b/pom.xml
@@ -219,8 +219,17 @@
             <configLocation>${checkstyle.dir}/kerby-checkstyle.xml</configLocation>
             <includeTestSourceDirectory>true</includeTestSourceDirectory>
             <encoding>UTF-8</encoding>
-            <failOnViolation>false</failOnViolation>
+            <failOnViolation>true</failOnViolation>
           </configuration>
+          <executions>
+            <execution>
+              <id>validate</id>
+              <phase>validate</phase>
+              <goals>
+                <goal>check</goal>
+              </goals>
+            </execution>
+          </executions>
         </plugin>
 
       </plugins>
@@ -275,7 +284,7 @@
       </properties>
     </profile>
     <profile>
-      <id>activate-in-module</id>
+      <id>activate-checkstyle-in-module</id>
       <activation>
         <file>
           <exists>${basedir}/../docs/kerby-checkstyle.xml</exists>
@@ -286,7 +295,7 @@
       </properties>
     </profile>
     <profile>
-      <id>activate-in-submodule</id>
+      <id>activate-checkstyle-in-submodule</id>
       <activation>
         <file>
           <exists>${basedir}/../../docs/kerby-checkstyle.xml</exists>
@@ -296,7 +305,6 @@
         <checkstyle.dir>${basedir}/../../docs</checkstyle.dir>
       </properties>
     </profile>
-
     <profile>
       <id>activate-checkstyle-supression</id>
       <activation>