Skip to content

ci: add PR coverage check workflow#8

Open
vividctrlalt wants to merge 10 commits intodevelopfrom
ci/coverage-check
Open

ci: add PR coverage check workflow#8
vividctrlalt wants to merge 10 commits intodevelopfrom
ci/coverage-check

Conversation

@vividctrlalt
Copy link
Owner

Summary

  • Add a new GitHub Actions workflow (coverage-check.yml) that compares code coverage between PR branch and base branch
  • Runs JaCoCo test coverage on both branches in parallel, then compares results
  • Posts a coverage report as a PR comment with per-module breakdown
  • Fails the check if line coverage drops more than the configurable threshold (default: 1%)

Test plan

  • Open a test PR against develop to verify the workflow triggers
  • Verify JaCoCo reports are generated and uploaded as artifacts
  • Verify the coverage comparison comment is posted on the PR
  • Verify the check fails when coverage drops beyond threshold

halibobo1205 and others added 3 commits March 3, 2026 16:00
Removed 4 config files from framework/src/main/resources that were
not referenced anywhere in the codebase:
- config-backup.conf: no references
- config-beta.conf: no references
- config-test-net.conf: no references
- config-localtest.conf: only used in tests, shadowed by
  src/test/resources/config-localtest.conf on the test classpath
@github-actions
Copy link

github-actions bot commented Mar 5, 2026

📊 Code Coverage Report

Overall Coverage

Metric Base PR Diff
Line 70.53% 70.53% ⚪ -0.00%
Branch 54.60% 54.56% ⚪ -0.04%
Instruction 68.73% 68.71% ⚪ -0.02%
Method 78.87% 78.91% ⚪ +0.05%
📦 Per-Module Coverage (Line)
Module Base PR Diff
framework 69.95% 69.94% ⚪ -0.00%
plugins 79.83% 79.83% ⚪ +0.00%

Note

Line coverage decreased by -0.00%, within the allowed threshold.

vividcoder and others added 7 commits March 5, 2026 18:06
Reduces Gradle startup overhead by running both tasks in one command.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Switch from pull_request to pull_request_target event so the workflow
token has write permission to post PR comments on fork PRs. Add
repository and ref parameters to checkout the PR head code correctly.
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