Skip to content

Commit

Permalink
Try adding code coverage for woodstox too
Browse files Browse the repository at this point in the history
  • Loading branch information
cowtowncoder committed Jul 11, 2021
1 parent 971aaa7 commit 12c56fa
Show file tree
Hide file tree
Showing 2 changed files with 35 additions and 5 deletions.
10 changes: 10 additions & 0 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,3 +36,13 @@ jobs:
${{ runner.os }}-maven-
- name: Build
run: ./mvnw -V -B -ff -ntp verify
- name: Generate code coverage
if: github.event_name != 'pull_request' && matrix.java_version == '8'
run: ./mvnw -B -ff -ntp test
- name: Publish code coverage
if: github.event_name != 'pull_request' && matrix.java_version == '8'
uses: codecov/codecov-action@v1
with:
token: ${{ secrets.CODECOV_TOKEN }}
file: ./target/site/jacoco/jacoco.xml
flags: unittests
30 changes: 25 additions & 5 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -162,15 +162,35 @@ SAX2 and Stax2 APIs
</dependencies>

<build>
<plugins>
<plugins>
<!-- 10-Jul-2021, tatu: Add code coverage again -->
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<executions>
<execution>
<goals>
<goal>prepare-agent</goal>
</goals>
</execution>
<!-- attached to Maven test phase -->
<execution>
<id>report</id>
<phase>test</phase>
<goals>
<goal>report</goal>
</goals>
</execution>
</executions>
</plugin>

<!-- 12-Oct-2019, tatu: Copied from
<!-- 12-Oct-2019, tatu: Copied from
https://github.com/stephenc/git-timestamp-maven-plugin/blob/master/pom.xml#L327-L337
to simplify releases. I hope.
-->
<plugin>
-->
<plugin>
<groupId>org.sonatype.plugins</groupId>
<artifactId>nexus-staging-maven-plugin</artifactId>
<version>1.6.6</version>
Expand All @@ -181,7 +201,7 @@ SAX2 and Stax2 APIs
<!-- 12-Oct-2019, tatu: Found this from output of mvn:relase -->
<stagingProfileId>b34f19b9cc6224</stagingProfileId>
</configuration>
</plugin>
</plugin>

<plugin>
<artifactId>maven-compiler-plugin</artifactId>
Expand Down

0 comments on commit 12c56fa

Please sign in to comment.