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 2f0c96c..4819189 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>

©2013 OpenDaylight, A Linux Foundation Collaborative Project. All Rights Reserved.
OpenDaylight is a registered trademark of The OpenDaylight Project, Inc.
Linux Foundation and OpenDaylight are registered trademarks of the Linux Foundation.
Linux is a registered trademark of Linus Torvalds.