Skip to content

Commit

Permalink
Switch from exclusion to inclusion for jacoco and reduce scope for ja…
Browse files Browse the repository at this point in the history
…va-checks
  • Loading branch information
Wohops committed Dec 21, 2015
1 parent 339f47b commit b617afc
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 20 deletions.
10 changes: 10 additions & 0 deletions java-checks/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,16 @@
</dependencies>
<build>
<plugins>
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<configuration>
<includes>
<include>org.sonar.java.checks.*</include>
<include>org.sonar.java.tag.*</include>
</includes>
</configuration>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-dependency-plugin</artifactId>
Expand Down
35 changes: 15 additions & 20 deletions java-squid/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -71,27 +71,22 @@
</dependencies>

<build>
<pluginManagement>
<plugins>
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<version>${jacoco.version}</version>
<configuration>
<excludes>
<!-- sources outside java-squid -->
<exclude>org.sonar.api.*</exclude>
<exclude>org.sonar.graph.*</exclude>
<exclude>org.sonar.squidbridge.*</exclude>
<exclude>org.sonar.sslr.*</exclude>
<!-- sources generated by java-maven-model -->
<exclude>org.sonar.maven.model.*</exclude>
</excludes>
</configuration>
</plugin>
</plugins>
</pluginManagement>
<plugins>
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<configuration>
<includes>
<include>org.sonar.java.*</include>
<include>org.sonar.maven.*</include>
<include>org.sonar.plugins.java.*</include>
</includes>
<excludes>
<!-- sources generated by java-maven-model -->
<exclude>org.sonar.maven.model.*</exclude>
</excludes>
</configuration>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-dependency-plugin</artifactId>
Expand Down

0 comments on commit b617afc

Please sign in to comment.