Skip to content

Bump jacoco and simplify code coverage CI#2963

Open
dplyukhin wants to merge 2 commits into
mainfrom
fix-temporal-sdk-java-main-code-coverage
Open

Bump jacoco and simplify code coverage CI#2963
dplyukhin wants to merge 2 commits into
mainfrom
fix-temporal-sdk-java-main-code-coverage

Conversation

@dplyukhin

Copy link
Copy Markdown
Contributor

What was changed

  • Bump JaCoCo to 0.8.15 for Java 23 compatibility
  • Avoid feeding duplicate classes to jacoco
  • Simplify code coverage CI workflow

Why?

Code coverage is failing in CI

Checklist

  1. Closes Code coverage causing commit failures on main #2962

  2. How was this tested:

  1. Any docs updates needed?

Comment on lines +6 to +15
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'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll revert this file after we've verified that code coverage is working properly

@dplyukhin
dplyukhin marked this pull request as ready for review July 23, 2026 21:53
@dplyukhin
dplyukhin requested a review from a team as a code owner July 23, 2026 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Code coverage causing commit failures on main

1 participant