Skip to content

Edburns/fix jacoco failure on main#1497

Merged
edburns merged 1 commit into
mainfrom
edburns/fix-jacoco-failure-on-main
May 29, 2026
Merged

Edburns/fix jacoco failure on main#1497
edburns merged 1 commit into
mainfrom
edburns/fix-jacoco-failure-on-main

Conversation

@edburns
Copy link
Copy Markdown
Collaborator

@edburns edburns commented May 29, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 29, 2026 18:17
@edburns edburns requested a review from a team as a code owner May 29, 2026 18:17
… and caused the failure

Re-enable Java JaCoCo badge update steps

On branch edburns/fix-jacoco-failure-on-main
modified:   .github/workflows/java-sdk-tests.yml

- Follow convention and invoke script via `bash` rather than assuming the execute bit is set.
@edburns edburns force-pushed the edburns/fix-jacoco-failure-on-main branch from 82d2eb9 to c6e05e4 Compare May 29, 2026 18:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Re-enables the Java CI workflow steps that generate and open an automated PR for updated JaCoCo coverage badge(s) on main, and adds a dedicated Bash script to build SVG badges from the JaCoCo CSV report.

Changes:

  • Enable JaCoCo badge generation + automated PR creation on successful main runs (JDK 25 matrix leg only).
  • Add .github/scripts/generate-java-coverage-badge.sh to compute overall/handwritten/generated coverage and emit SVG badges.
Show a summary per file
File Description
.github/workflows/java-sdk-tests.yml Turns the JaCoCo badge + PR steps back on for main and ensures the badge script is invoked via bash.
.github/scripts/generate-java-coverage-badge.sh New script that parses JaCoCo CSV totals and generates SVG badge files under .github/badges/.

Copilot's findings

  • Files reviewed: 1/2 changed files
  • Comments generated: 0

@edburns edburns enabled auto-merge May 29, 2026 18:21
@edburns edburns disabled auto-merge May 29, 2026 18:21
@edburns edburns enabled auto-merge May 29, 2026 18:21
@edburns edburns disabled auto-merge May 29, 2026 18:22
@edburns edburns merged commit 5e9d02e into main May 29, 2026
14 of 16 checks passed
@edburns edburns deleted the edburns/fix-jacoco-failure-on-main branch May 29, 2026 18:23
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