diff options
author | Pamela Dragosh <pdragosh@research.att.com> | 2018-09-25 09:20:32 -0400 |
---|---|---|
committer | Pamela Dragosh <pdragosh@research.att.com> | 2018-09-25 11:03:11 -0400 |
commit | 8218ae0be874fe0a662f52ac74cd43f564e7b666 (patch) | |
tree | 5c1603e6d16af60b4280e25deff759b2a02fb77f /pom.xml | |
parent | da9e01a78e3205ee7238196a574adcf11c041797 (diff) |
Clean up checkstyle declaration
Fixed the path for finding the src files.
Added eclipse lifecycle to ignore the oparent checkstyle
bug.
Cleaned up existing checkstyle issues.
Added entries into .gitignore to ignore debug logging
directories created.
Issue-ID: POLICY-1136
Change-Id: I684038babcb9723cfc4bef2b2a713425ae077599
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
Diffstat (limited to 'pom.xml')
-rw-r--r-- | pom.xml | 49 |
1 files changed, 34 insertions, 15 deletions
@@ -27,12 +27,11 @@ <groupId>org.onap.policy.parent</groupId> <artifactId>integration</artifactId> <version>2.0.0-SNAPSHOT</version> - <relativePath/> + <relativePath /> </parent> <groupId>org.onap.policy.api</groupId> <artifactId>policy-api</artifactId> - <version>2.0.0-SNAPSHOT</version> <packaging>pom</packaging> @@ -41,8 +40,7 @@ <properties> <!-- sonar/jacoco overrides --> - <!-- Overriding oparent default sonar/jacoco settings Combine all our reports - into one file shared across sub-modules --> + <!-- Overriding oparent default sonar/jacoco settings Combine all our reports into one file shared across sub-modules --> <sonar.jacoco.reportPath>${project.basedir}/../target/code-coverage/jacoco-ut.exec</sonar.jacoco.reportPath> <sonar.jacoco.itReportPath>${project.basedir}/../target/code-coverage/jacoco-it.exec</sonar.jacoco.itReportPath> <sonar.dynamicAnalysis>reuseReports</sonar.dynamicAnalysis> @@ -124,12 +122,11 @@ </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 --> + <!-- 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 --> - <sourceDirectory>${project.build.sourceDirectory}/src/main/java</sourceDirectory> + <sourceDirectory>${project.build.sourceDirectory}</sourceDirectory> <includeResources>true</includeResources> <includeTestSourceDirectory>true</includeTestSourceDirectory> <includeTestResources>true</includeTestResources> @@ -145,7 +142,7 @@ <dependency> <groupId>org.onap.oparent</groupId> <artifactId>checkstyle</artifactId> - <version>1.1.0</version> + <version>${oparent.version}</version> <scope>compile</scope> </dependency> </dependencies> @@ -158,8 +155,7 @@ <artifactId>jacoco-maven-plugin</artifactId> <version>${jacoco.version}</version> <configuration> - <!-- Note: This exclusion list should match <sonar.exclusions> property - above --> + <!-- Note: This exclusion list should match <sonar.exclusions> property above --> <excludes> <exclude>**/gen/**</exclude> <exclude>**/generated-sources/**</exclude> @@ -168,8 +164,8 @@ </excludes> </configuration> <executions> - <!-- Prepares the property pointing to the JaCoCo runtime agent which - is passed as VM argument when Maven the Surefire plugin is executed. --> + <!-- Prepares the property pointing to the JaCoCo runtime agent which is passed as VM argument when + Maven the Surefire plugin is executed. --> <execution> <id>pre-unit-test</id> <goals> @@ -179,8 +175,7 @@ <destFile>${sonar.jacoco.reportPath}</destFile> </configuration> </execution> - <!-- Ensures that the code coverage report for unit tests is created - after unit tests have been run. --> + <!-- Ensures that the code coverage report for unit tests is created after unit tests have been run. --> <execution> <id>post-unit-test</id> <phase>test</phase> @@ -193,6 +188,30 @@ </execution> </executions> </plugin> + <plugin> + <groupId>org.eclipse.m2e</groupId> + <artifactId>lifecycle-mapping</artifactId> + <version>1.0.0</version> + <configuration> + <lifecycleMappingMetadata> + <pluginExecutions> + <pluginExecution> + <pluginExecutionFilter> + <groupId>org.apache.maven.plugins</groupId> + <artifactId>maven-checkstyle-plugin</artifactId> + <versionRange>2.17,)</versionRange> + <goals> + <goal>check</goal> + </goals> + </pluginExecutionFilter> + <action> + <ignore /> + </action> + </pluginExecution> + </pluginExecutions> + </lifecycleMappingMetadata> + </configuration> + </plugin> </plugins> </pluginManagement> </build> |