diff options
| author |   Pamela Dragosh <pdragosh@research.att.com> | 2018-09-21 20:08:11 -0400 |
|---|---|---|
| committer | Pamela Dragosh <pdragosh@research.att.com> | 2018-09-24 15:30:14 -0400 |
| commit | a24a985d328b97f50a41d3448d235866af625ca9 (patch) | |
| tree | 06c95950000fc2b512ec1e4da5aded2c5bc84e78 /controlloop/common/policy-yaml | |
| parent | 3c52de0725c942fe1f2fdeb05c8960645bc66f01 (diff) | |
Fix checkstyle declarations
I fixed what I thought would be tolerable for this release.
Same issue as policy/common I had to defined the checkstyle
in each repo where the suppressions were needed.
Issue-ID: POLICY-1135
Change-Id: I8f30bee7e9cddc692ddad3cf88acedb2e6b4781b
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
Diffstat (limited to 'controlloop/common/policy-yaml')
| -rw-r--r-- | controlloop/common/policy-yaml/pom.xml | 49 |
1 files changed, 49 insertions, 0 deletions
diff --git a/controlloop/common/policy-yaml/pom.xml b/controlloop/common/policy-yaml/pom.xml index 633bd9c4f..b48e74d67 100644 --- a/controlloop/common/policy-yaml/pom.xml +++ b/controlloop/common/policy-yaml/pom.xml @@ -79,4 +79,53 @@ <scope>test</scope> </dependency> </dependencies> + + <!-- + Without this defined here, the checkstyle plugin just simply fails outright. This needs to be investigated + further as there was no real debugging details as to why or any configuration issue detected. + NOTE: I had to append src/main/java to the sourceDirectory. + Is it a reactor ordering issue? I'm wondering why it is being compiled after aai, in between the sub-modules + in model-impl. Dependency ordering?? Circular dependency?? + --> + <build> + <plugins> + <plugin> + <artifactId>maven-checkstyle-plugin</artifactId> + <executions> + <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 --> + <sourceDirectory>${project.build.sourceDirectory}/src/main/java</sourceDirectory> + <includeResources>true</includeResources> + <includeTestSourceDirectory>true</includeTestSourceDirectory> + <includeTestResources>true</includeTestResources> + <excludes> + </excludes> + <consoleOutput>true</consoleOutput> + <failsOnViolation>true</failsOnViolation> + <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> |