mirror of https://github.com/apache/jclouds.git
Fail build on checkstyle warnings
This commit is contained in:
parent
d6eb54e2b4
commit
9384575d1e
|
@ -84,7 +84,7 @@
|
||||||
<url>https://git-wip-us.apache.org/repos/asf?p=jclouds.git</url>
|
<url>https://git-wip-us.apache.org/repos/asf?p=jclouds.git</url>
|
||||||
<tag>HEAD</tag>
|
<tag>HEAD</tag>
|
||||||
</scm>
|
</scm>
|
||||||
|
|
||||||
<repositories>
|
<repositories>
|
||||||
<repository>
|
<repository>
|
||||||
<id>apache-snapshots</id>
|
<id>apache-snapshots</id>
|
||||||
|
@ -340,7 +340,7 @@
|
||||||
<arguments>-Pdoc -Papache-release ${arguments}</arguments>
|
<arguments>-Pdoc -Papache-release ${arguments}</arguments>
|
||||||
</configuration>
|
</configuration>
|
||||||
</plugin>
|
</plugin>
|
||||||
|
|
||||||
<plugin>
|
<plugin>
|
||||||
<groupId>org.apache.rat</groupId>
|
<groupId>org.apache.rat</groupId>
|
||||||
<artifactId>apache-rat-plugin</artifactId>
|
<artifactId>apache-rat-plugin</artifactId>
|
||||||
|
@ -361,7 +361,7 @@
|
||||||
<exclude>**/src/test/resources/**/*.txt</exclude>
|
<exclude>**/src/test/resources/**/*.txt</exclude>
|
||||||
<exclude>**/src/test/resources/**/*.gz</exclude>
|
<exclude>**/src/test/resources/**/*.gz</exclude>
|
||||||
<exclude>**/src/test/resources/**/*.xml</exclude>
|
<exclude>**/src/test/resources/**/*.xml</exclude>
|
||||||
|
|
||||||
<!-- META-INF/services files -->
|
<!-- META-INF/services files -->
|
||||||
<exclude>**/services/*LoggingModule</exclude>
|
<exclude>**/services/*LoggingModule</exclude>
|
||||||
<exclude>**/services/*ApiMetadata</exclude>
|
<exclude>**/services/*ApiMetadata</exclude>
|
||||||
|
@ -671,10 +671,25 @@
|
||||||
<plugin>
|
<plugin>
|
||||||
<artifactId>maven-checkstyle-plugin</artifactId>
|
<artifactId>maven-checkstyle-plugin</artifactId>
|
||||||
<version>2.12.1</version>
|
<version>2.12.1</version>
|
||||||
|
<executions>
|
||||||
|
<execution>
|
||||||
|
<id>checkstyle</id>
|
||||||
|
<phase>validate</phase>
|
||||||
|
<goals>
|
||||||
|
<goal>check</goal>
|
||||||
|
</goals>
|
||||||
|
<configuration>
|
||||||
|
<failOnViolation>true</failOnViolation>
|
||||||
|
</configuration>
|
||||||
|
</execution>
|
||||||
|
</executions>
|
||||||
<configuration>
|
<configuration>
|
||||||
<!-- configLocation configured via profiles -->
|
<!-- configLocation configured via profiles -->
|
||||||
<violationSeverity>warning</violationSeverity>
|
<violationSeverity>warning</violationSeverity>
|
||||||
<includeTestSourceDirectory>true</includeTestSourceDirectory>
|
<includeTestSourceDirectory>true</includeTestSourceDirectory>
|
||||||
|
<failOnViolation>true</failOnViolation>
|
||||||
|
<failsOnError>true</failsOnError>
|
||||||
|
<violationSeverity>warning</violationSeverity>
|
||||||
</configuration>
|
</configuration>
|
||||||
</plugin>
|
</plugin>
|
||||||
</plugins>
|
</plugins>
|
||||||
|
@ -947,7 +962,7 @@
|
||||||
<configuration>
|
<configuration>
|
||||||
<descriptors>
|
<descriptors>
|
||||||
<descriptor>src-descriptor.xml</descriptor>
|
<descriptor>src-descriptor.xml</descriptor>
|
||||||
<descriptor>provided-dependencies-descriptor.xml</descriptor>
|
<descriptor>provided-dependencies-descriptor.xml</descriptor>
|
||||||
<descriptor>jar-with-dependencies-no-core-no-apis-descriptor.xml</descriptor>
|
<descriptor>jar-with-dependencies-no-core-no-apis-descriptor.xml</descriptor>
|
||||||
</descriptors>
|
</descriptors>
|
||||||
</configuration>
|
</configuration>
|
||||||
|
@ -963,7 +978,7 @@
|
||||||
</plugin>
|
</plugin>
|
||||||
</plugins>
|
</plugins>
|
||||||
</build>
|
</build>
|
||||||
</profile>
|
</profile>
|
||||||
<profile>
|
<profile>
|
||||||
<id>site</id>
|
<id>site</id>
|
||||||
<build>
|
<build>
|
||||||
|
@ -1019,7 +1034,18 @@
|
||||||
<artifactId>maven-checkstyle-plugin</artifactId>
|
<artifactId>maven-checkstyle-plugin</artifactId>
|
||||||
<configuration>
|
<configuration>
|
||||||
<configLocation>../resources/checkstyle.xml</configLocation>
|
<configLocation>../resources/checkstyle.xml</configLocation>
|
||||||
|
<failOnViolation>true</failOnViolation>
|
||||||
|
<failsOnError>true</failsOnError>
|
||||||
|
<violationSeverity>warning</violationSeverity>
|
||||||
</configuration>
|
</configuration>
|
||||||
|
<executions>
|
||||||
|
<execution>
|
||||||
|
<phase>validate</phase>
|
||||||
|
<goals>
|
||||||
|
<goal>check</goal>
|
||||||
|
</goals>
|
||||||
|
</execution>
|
||||||
|
</executions>
|
||||||
</plugin>
|
</plugin>
|
||||||
</plugins>
|
</plugins>
|
||||||
</build>
|
</build>
|
||||||
|
@ -1046,9 +1072,20 @@
|
||||||
<configuration>
|
<configuration>
|
||||||
<!-- jclouds-resources has the checkstyle config in the classpath -->
|
<!-- jclouds-resources has the checkstyle config in the classpath -->
|
||||||
<configLocation>resources/checkstyle.xml</configLocation>
|
<configLocation>resources/checkstyle.xml</configLocation>
|
||||||
|
<failOnViolation>true</failOnViolation>
|
||||||
|
<failsOnError>true</failsOnError>
|
||||||
|
<violationSeverity>warning</violationSeverity>
|
||||||
</configuration>
|
</configuration>
|
||||||
|
<executions>
|
||||||
|
<execution>
|
||||||
|
<phase>validate</phase>
|
||||||
|
<goals>
|
||||||
|
<goal>check</goal>
|
||||||
|
</goals>
|
||||||
|
</execution>
|
||||||
|
</executions>
|
||||||
</plugin>
|
</plugin>
|
||||||
</plugins>
|
</plugins>
|
||||||
</build>
|
</build>
|
||||||
</profile>
|
</profile>
|
||||||
<profile>
|
<profile>
|
||||||
|
|
Loading…
Reference in New Issue