diff options
author | liamfallon <liam.fallon@est.tech> | 2020-06-18 11:45:14 +0100 |
---|---|---|
committer | liamfallon <liam.fallon@est.tech> | 2020-06-18 14:06:41 +0100 |
commit | 324700e31ee692899c1658bf647963ca04ffa425 (patch) | |
tree | 5c424e016122dc667be18a8f8090bd25991961b1 | |
parent | c8345b4f8e49a21f02f784dfdacea420422f5c20 (diff) |
Consolidate checkstyle plugin into policy parent
At the moment, the checkstyle plugin definitions are repeated separately
in all the policy components. This review adds the checkstyle plugin
defintions into policy parent.
Subsequent reviews will remove it from the individual component repos.
Issue-ID: POLICY-2188
Change-Id: I7e786e82086d31df8597c08382bfe839e60074e3
Signed-off-by: liamfallon <liam.fallon@est.tech>
-rw-r--r-- | integration/pom.xml | 59 |
1 files changed, 57 insertions, 2 deletions
diff --git a/integration/pom.xml b/integration/pom.xml index 2ee3aa92..2c28475e 100644 --- a/integration/pom.xml +++ b/integration/pom.xml @@ -2,7 +2,7 @@ ============LICENSE_START======================================================= Copyright (C) 2018 Ericsson. All rights reserved. Modifications Copyright (C) 2018-2020 AT&T. All rights reserved. - Modifications Copyright (C) 2019 Nordix Foundation. + Modifications Copyright (C) 2019-2020 Nordix Foundation. Modifications Copyright (C) 2020 Bell Canada. ================================================================================ Licensed under the Apache License, Version 2.0 (the "License"); @@ -719,7 +719,62 @@ </execution> </executions> </plugin> + <plugin> + <artifactId>maven-checkstyle-plugin</artifactId> + <executions> + <execution> + <id>onap-license</id> + <goals> + <goal>check</goal> + </goals> + <phase>process-sources</phase> + <configuration> + <configLocation>onap-checkstyle/check-license.xml</configLocation> + <includeResources>false</includeResources> + <includeTestSourceDirectory>true</includeTestSourceDirectory> + <includeTestResources>false</includeTestResources> + <sourceDirectories> + <sourceDirectory>${project.build.sourceDirectory}</sourceDirectory> + </sourceDirectories> + <consoleOutput>true</consoleOutput> + <failOnViolation>true</failOnViolation> + <violationSeverity>warning</violationSeverity> + </configuration> + </execution> + <execution> + <id>onap-java-style</id> + <goals> + <goal>check</goal> + </goals> + <phase>process-sources</phase> + <configuration> + <!-- Use Google Java Style Guide: + https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/google_checks.xml + with minor changes --> + <configLocation>onap-checkstyle/onap-java-style.xml</configLocation> + <!-- <sourceDirectory> is needed so that checkstyle ignores the generated + sources directory --> + <sourceDirectories>${project.build.sourceDirectory}</sourceDirectories> + <includeResources>true</includeResources> + <includeTestSourceDirectory>true</includeTestSourceDirectory> + <includeTestResources>true</includeTestResources> + <excludes> + </excludes> + <consoleOutput>true</consoleOutput> + <failOnViolation>true</failOnViolation> + <violationSeverity>warning</violationSeverity> + </configuration> + </execution> + </executions> + <dependencies> + <dependency> + <groupId>org.onap.oparent</groupId> + <artifactId>checkstyle</artifactId> + <version>${oparent.version}</version> + <scope>compile</scope> + </dependency> + </dependencies> + </plugin> </plugins> </build> - </project> |