Fix adding coverage to PRs (getindata#131)

hyperpaas · Jan 21, 2025 · d01a508 · d01a508
1 parent f2176e1
commit d01a508
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -42,7 +42,7 @@ jobs:
 
       - name: Add coverage to PR
         id: jacoco
-        uses: madrapps/jacoco-report@v1.2
+        uses: madrapps/jacoco-report@v1.7.1
         with:
           paths: ${{ github.workspace }}/target/site/jacoco/jacoco.xml
           token: ${{ secrets.GITHUB_TOKEN }}

diff --git a/pom.xml b/pom.xml
@@ -83,7 +83,7 @@ under the License.
     <assertj.core.version>3.21.0</assertj.core.version>
     <mockito.version>4.0.0</mockito.version>
     <wiremock.version>2.27.2</wiremock.version>
-    <jacoco.plugin.version>0.8.7</jacoco.plugin.version>
+    <jacoco.plugin.version>0.8.12</jacoco.plugin.version>
     <maven.shade.plugin.version>3.1.1</maven.shade.plugin.version>
     <mockito-inline.version>4.6.1</mockito-inline.version>
   </properties>
@@ -298,7 +298,7 @@ under the License.
         <version>3.0.0-M5</version>
         <configuration>
           <!-- argLine needed for Flink 1.16 and 1.17 or there are unit test errors-->
-          <argLine>--add-opens java.base/java.util=ALL-UNNAMED --add-opens java.base/java.lang=ALL-UNNAMED</argLine>
+          <argLine>@{argLine} --add-opens java.base/java.util=ALL-UNNAMED --add-opens java.base/java.lang=ALL-UNNAMED</argLine>
         </configuration>
       </plugin>