Skip to content

Update JaCoCo coverage badge#226

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
auto/update-jacoco-badge
Open

Update JaCoCo coverage badge#226
github-actions[bot] wants to merge 1 commit into
mainfrom
auto/update-jacoco-badge

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Automated JaCoCo coverage badge update from CI.

Copy link
Copy Markdown
Collaborator

@edburns edburns left a comment

Choose a reason for hiding this comment

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

@copilot whoa. Why the dramatic drop in code coverage? There was a huge drop in generated. Also, why the re-added "merged" badge? We broke out the badges into two buckets a while ago.

Copy link
Copy Markdown
Contributor

Copilot AI commented May 22, 2026

@copilot whoa. Why the dramatic drop in code coverage? There was a huge drop in generated. Also, why the re-added "merged" badge? We broke out the badges into two buckets a while ago.

Addressed in 326dac6.

  • I removed merged badge generation and deletion is now enforced in the badge script, so only handwritten/generated badges are produced.
  • The generated percentage drop is from current JaCoCo data (not a badge calc bug): generated coverage is now being computed over com.github.copilot.sdk.generated*, and many newly generated RPC/event classes are currently at 0% instruction coverage, which pulls the aggregate down.

Copilot AI requested a review from edburns May 22, 2026 21:49
@github-actions github-actions Bot force-pushed the auto/update-jacoco-badge branch 14 times, most recently from 984ac51 to 8ed8ae5 Compare May 28, 2026 23:44
@github-actions github-actions Bot force-pushed the auto/update-jacoco-badge branch from 8ed8ae5 to 13225ca Compare May 28, 2026 23:44
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.

2 participants