Repository navigation
Add coverage report as a PR check run - #386
Open
dmarcelino wants to merge 3 commits into
Open
dmarcelino wants to merge 3 commits into
dmarcelino wants to merge 3 commits into
Conversation
- New inputs: add-check, fail-check-below-threshold; comment-type gains none - Check title shows overall coverage with a pass/fail symbol; body reuses the comment markdown - Missing checks: write permission fails the action regardless of continue-on-error - skip-if-no-changes only suppresses comment and summary, the check is still published - Split action.ts into inputs, status and publish modules - Mask token in debug-mode input logging - Add tslib dev dependency required by the rollup typescript plugin; rebuild dist
dmarcelino
marked this pull request as ready for review
September 24, 2026 21:18
# Conflicts: # package-lock.json # package.json
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi @thsaravana , thanks for the jacoco-report! It has been helpful. Recently I've found the need to have it as PR check instead of a comment, hence this PR. Review it and do let me know if you need any changes.
Summary
Publishes the coverage report as a GitHub check run on the head commit, in addition to (or instead of) the PR comment / workflow summary.
New inputs
add-check(defaultfalse): creates a check run named aftertitle(fallbackJaCoCo Report). The check title shows the overall coverage plus the delta when coverage dropped, e.g.Overall 69.09% (-1.07%); the check body reuses the existing comment markdown. Requires thechecks: writepermission.fail-check-below-threshold(defaultfalse): marks the check as failed when overall or changed-lines coverage is below the configured minimum. Requiresadd-check.comment-typegains anoneoption (must be combined withadd-check: true, otherwise there is nothing to publish).Behaviour notes
checks: writepermission (403 from the API) fails the action regardless ofcontinue-on-error, with a message telling the user which permission to add.skip-if-no-changesonly suppresses the comment and workflow summary; the check run is still published.Refactor
action.tsintosrc/inputs.ts.src/status.ts; comment / summary / check publishing split intosrc/publish/{comment,summary,check}.ts.Tests / build / docs
inputs,status,publish_check,action_check.tslibadded as a dev dependency (required by the rollup typescript plugin);dist/rebuilt.checks: writepermission, updated example, and two check-run screenshots.Note for reviewers:
.gitignorealso gains.codegraph(a local tooling ignore). Happy to drop it if you'd rather not have it upstream.