Skip to content

Add coverage artifact upload and enforcement#14

Merged
tcheeric merged 2 commits intodevelopfrom
codex/add-jacoco-report-upload-step
Aug 9, 2025
Merged

Add coverage artifact upload and enforcement#14
tcheeric merged 2 commits intodevelopfrom
codex/add-jacoco-report-upload-step

Conversation

@tcheeric
Copy link
Copy Markdown
Collaborator

@tcheeric tcheeric commented Aug 9, 2025

Summary

  • add GitHub Actions workflow to build, check Jacoco coverage, and upload the aggregated HTML report
  • enforce 80% instruction coverage via Jacoco check rule

Testing

  • mvn -q verify
  • mvn -q jacoco:check@coverage-check

https://chatgpt.com/codex/tasks/task_b_6896a3f837388331aa43b3560ebc7867

Copy link
Copy Markdown

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

This PR adds automated code coverage reporting and enforcement to the project by integrating JaCoCo coverage tooling with GitHub Actions CI pipeline. The changes establish an 80% instruction coverage threshold and automatically upload coverage reports as build artifacts.

  • Adds GitHub Actions CI workflow for automated build, test, and coverage validation
  • Enforces 80% instruction coverage requirement via JaCoCo Maven plugin configuration
  • Uploads aggregated HTML coverage reports as GitHub Actions artifacts

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
pom.xml Adds JaCoCo coverage check execution with 80% instruction coverage threshold
.github/workflows/ci.yml Creates new CI workflow for build automation and coverage artifact upload

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@tcheeric tcheeric merged commit eed9347 into develop Aug 9, 2025
1 check failed
@tcheeric tcheeric deleted the codex/add-jacoco-report-upload-step branch August 9, 2025 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants