Modify archetype to enable checkstyle failOnError by default 77/52677/2
authorRyan Goulding <ryandgoulding@gmail.com>
Thu, 2 Mar 2017 21:53:49 +0000 (16:53 -0500)
committerTom Pantelis <tpanteli@brocade.com>
Fri, 3 Mar 2017 13:02:28 +0000 (13:02 +0000)
As a community, we want to try our best to influence new projects to write
clean, consistent code.  As such, we really ought to enable checkstyle failOnError
for projects that are generated using the startup archetype.  This change does
that;  projects can still turn this off, but it requires manual opt out.

By default, we should really try to influence failOnError so people write
good looking code.

Change-Id: If740df92cbc3d8ff8a0169d323028415e239a15a
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
opendaylight/archetypes/opendaylight-startup/src/main/resources/archetype-resources/pom.xml

index 2f0c96c278876e6e489cf556acc4bca98f3766f3..481918991aebe35ba51a6ff2c63a00ce191985c1 100644 (file)
@@ -87,6 +87,12 @@ and is available at http://www.eclipse.org/legal/epl-v10.html INTERNAL
             </execution>
             </executions>
           </plugin>
+          <plugin>
+            <artifactId>maven-checkstyle-plugin</artifactId>
+            <configuration>
+              <failsOnError>true</failsOnError>
+            </configuration>
+          </plugin>
         </plugins>
       </build>
     </profile>