Skip to content

Replace badge workflows with a single maven build workflow - #19

Merged
lpenap merged 1 commit into
masterfrom
lpenap/jacoco-workflow
Sep 16, 2026
Merged

lpenap merged 1 commit into
masterfrom
lpenap/jacoco-workflow

Conversation

@lpenap

@lpenap lpenap commented Sep 16, 2026

Copy link
Copy Markdown
Owner

Summary

Ports the maven.yml workflow from java-monitor-example and removes merged_master.yml and pull_request.yml.

  • Triggers on push to master and on every pull request, so master is checked out explicitly instead of relying on the closed-PR merge ref
  • Badge generation, coverage logging and the dependency graph update are gated on master pushes only
  • Adds id: jacoco so the coverage log step actually prints the values (it was printing empty strings)
  • Declares contents: write and pull-requests: write on the job instead of relying on the repo default
  • Commits the regenerated badge through a short-lived ci/jacoco-badge-* branch and auto-merged PR instead of pushing to master directly
  • Uses ./mvnw -B -ntp verify for the build
  • README build badge now points at maven.yml

Verification

  • Reviewed the last two merged_master.yml runs: all steps green, badge unchanged
  • Built master locally: 93.9% instruction and 80% branch coverage, matching the committed badges
  • advanced-security/maven-dependency-submission-action@v6.0.1 published 2026-09-04

🤖 Generated with Claude Code

Merge merged_master.yml and pull_request.yml into maven.yml, modelled on
java-monitor-example. Builds run on pushes to master and on every pull
request; badge generation, coverage logging and the dependency graph
update run on master pushes only. The badge is committed through a
short-lived branch and pull request instead of a direct push, the
coverage log step now reads the generator outputs via its step id, and
the README build badge points at the new workflow.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@lpenap
lpenap merged commit 50fb2c8 into master Sep 16, 2026
3 checks passed
@lpenap
lpenap deleted the lpenap/jacoco-workflow branch September 16, 2026 15:48
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.

1 participant