Skip to content

Commit

Permalink
Cobertura did not support java 8 lambda.
Browse files Browse the repository at this point in the history
So change coverage tools from Cobertura to JaCoCo
  • Loading branch information
peng-yongsheng committed Aug 1, 2017
1 parent 667c0f4 commit 638789a
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 20 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Expand Up @@ -18,5 +18,5 @@ script:
- mvn clean install

after_success:
- mvn clean cobertura:cobertura coveralls:report
- mvn clean test jacoco:report coveralls:report
- bash ./travis/push_image.sh
30 changes: 11 additions & 19 deletions pom.xml
Expand Up @@ -159,25 +159,17 @@
</plugin>
<!-- 覆盖率 -->
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>cobertura-maven-plugin</artifactId>
<version>2.7</version>
<configuration>
<check>true</check>
<encoding>UTF-8</encoding>
<aggregate>true</aggregate>
<formats>
<format>xml</format>
<format>html</format>
</formats>
<instrumentation>
<excludes>
<exclude>org/skywalking/apm/network/**/*.class</exclude>
<exclude>org/skywalking/apm/collector/remote/grpc/**/*.class</exclude>
<exclude>org/skywalking/apm/agent/core/context/ids/base64/*.class</exclude>
</excludes>
</instrumentation>
</configuration>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<version>0.7.9</version>
<executions>
<execution>
<id>prepare-agent</id>
<goals>
<goal>prepare-agent</goal>
</goals>
</execution>
</executions>
</plugin>
<plugin>
<groupId>com.spotify</groupId>
Expand Down

0 comments on commit 638789a

Please sign in to comment.