Repository navigation
ci: 👷 Enforce coverage and report test results on PRs - #73
Merged
Merged
Conversation
Contributor
|
Linux CI: success · Commit Test results (Linux)
Test coverage
Includes the test JVM and CLI subprocesses. Excludes optional real-Collector smoke tests. Workflow logs and report artifacts Updated on each CI run. Windows checks, when required, are reported separately in the workflow. |
oxspan
marked this pull request as ready for review
November 11, 2026 07:14
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
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.
Why
Passing tests previously provided no coverage report, minimum coverage check, or PR-visible result summary. Refs #72.
Changes
./gradlew -Pcoverage=true coverageCheckwith JaCoCo, enforcing 90% line and 80% branch coverage. Merge test JVM data with separate CLI subprocess execution files without changing CLI output.Validation
2337502, including the comment job. Published bot comment: 228 tests passed, 95.55% lines (2060/2156), 80.21% branches (1228/1531).coverageCheck verifyReleaseInputspassed; coverage-disabled command distribution tests passed (5 tests).actionlintand whitespace checks passed. Local act was attempted but could not fetchactions/checkoutdue to an authentication error; hosted CI completed successfully.Compatibility and risks
Instrumentation is test-only and opt-in locally. Normal Linux CI enables it; ordinary tests and Windows checks keep coverage disabled. Historical release-asset repair skips the new coverage gate. No production CLI options, output formats, runtime dependencies or capture formats change. Codecov comparisons and required merge checks remain follow-up work.