You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2022/11/27 16:57:18 UTC

[commons-bcel] branch master updated: Let coverage run on both JDK 8 and 11 (#176)

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git


The following commit(s) were added to refs/heads/master by this push:
     new a40ec0d4 Let coverage run on both JDK 8 and 11 (#176)
a40ec0d4 is described below

commit a40ec0d4aea093875c48caa89c3c0262931245b5
Author: nbauma109 <nb...@users.noreply.github.com>
AuthorDate: Sun Nov 27 17:57:13 2022 +0100

    Let coverage run on both JDK 8 and 11 (#176)
    
    Some execution paths are related to the use of modules. These execution
    paths can't be hit if coverage runs only on JDK 8.
---
 .github/workflows/coverage.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
index 71db7bfd..99a3bfc0 100644
--- a/.github/workflows/coverage.yml
+++ b/.github/workflows/coverage.yml
@@ -26,7 +26,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        java: [ 8 ]
+        java: [ 8, 11 ]
 
     steps:
     - uses: actions/checkout@v3.1.0
@@ -44,7 +44,7 @@ jobs:
         distribution: 'temurin'
         java-version: ${{ matrix.java }}
     - name: Build with Maven
-      run: mvn -V install jacoco:report --file pom.xml --no-transfer-progress
+      run: mvn -V install jacoco:report -DdataFile=jacoco_jdk${{ matrix.java }}.exec --file pom.xml --no-transfer-progress
 
     - name: Upload coverage to Codecov
       uses: codecov/codecov-action@v3