From ab258ce11baad182be8735261f6a03ab0d4bf1aa Mon Sep 17 00:00:00 2001 From: Dan Plyukhin Date: Thu, 23 Jul 2026 17:02:09 -0400 Subject: [PATCH 1/2] Bump jacoco and simplify code coverage CI --- .github/workflows/coverage.yml | 11 ++--------- gradle/jacoco.gradle | 14 ++++++++++++-- 2 files changed, 14 insertions(+), 11 deletions(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index ee48ae8188..dbdcdbd69d 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -25,14 +25,7 @@ jobs: - name: Set up Gradle uses: gradle/actions/setup-gradle@ac396bf1a80af16236baf54bd7330ae21dc6ece5 # v6 - - name: Run Tests - run: ./gradlew test -x spotlessCheck -x spotlessApply -Pjacoco -PtestJavaVersion=23 - continue-on-error: true - - - name: Run Test Coverage - run: ./gradlew testCodeCoverageReport -Pjacoco - - - name: Publish Coverage + - name: Run and Publish Test Coverage env: COVERALLS_REPO_TOKEN: ${{ secrets.COVERALLS_REPO_TOKEN }} - run: ./gradlew coverallsJacoco -Pjacoco + run: ./gradlew coverallsJacoco -x spotlessCheck -x spotlessApply -Pjacoco -PtestJavaVersion=23 diff --git a/gradle/jacoco.gradle b/gradle/jacoco.gradle index 43bf9075a8..6f71d5e4df 100644 --- a/gradle/jacoco.gradle +++ b/gradle/jacoco.gradle @@ -3,6 +3,12 @@ apply plugin: 'jacoco-report-aggregation' apply plugin: 'com.github.nbaztec.coveralls-jacoco' +def jacocoToolVersion = "0.8.15" + +jacoco { + toolVersion = jacocoToolVersion +} + dependencies { jacocoAggregation project(':temporal-kotlin') jacocoAggregation project(':temporal-opentracing') @@ -39,7 +45,11 @@ testCodeCoverageReport { } getClassDirectories().setFrom(files( - jacocoSubprojects.collect {it.fileTree(dir: "${it.buildDir}/classes", exclude: jacocoExclusions)} + jacocoSubprojects.collect { + it.sourceSets.main.output.classesDirs.asFileTree.matching { + exclude jacocoExclusions + } + } )) } @@ -57,7 +67,7 @@ subprojects { apply plugin: 'jacoco' jacoco { - toolVersion = "0.8.9" + toolVersion = jacocoToolVersion } jacocoTestReport { From bad325ef034bf41d4616f5a6dfc6de9aec706018 Mon Sep 17 00:00:00 2001 From: Dan Plyukhin Date: Thu, 23 Jul 2026 17:09:28 -0400 Subject: [PATCH 2/2] Temporarily enable code coverage on this PR --- .github/workflows/coverage.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index dbdcdbd69d..88dcad562a 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -3,12 +3,16 @@ on: push: branches: - main + pull_request: + branches: + - main permissions: contents: read jobs: code-coverage: + if: github.event_name == 'push' || github.head_ref == 'fix-temporal-sdk-java-main-code-coverage' runs-on: ubuntu-latest-16-cores steps: - name: Checkout