Skip to content

feat(ci): support consolidated reports from nested workflows - #590

Merged
neilime merged 1 commit into
mainfrom
feat/consolidated-action-coverage
Nov 10, 2026
Merged

neilime merged 1 commit into
mainfrom
feat/consolidated-action-coverage

Conversation

@neilime

@neilime neilime commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

Nested Node.js CI calls cannot currently disable individual comments through test options or associate their report artifacts with outer calling jobs. This prevents action test matrices from publishing a single combined coverage comment through Common's ci-reports workflow.

Forward test.comment and test.report-group to the test action. Use the outer job ID to group artifacts, label each artifact with its working directory, and retain unique names for matrix runs. Default reporting and explicit workflow permissions remain in place.

Validate the new options, regenerate action/workflow documentation with CI Dokumentor, and extend the workflow integration check to retrieve native coverage using the outer job ID. Consumers set comment: false and call ci-reports once after their producers, including failed jobs.

Validation: 67 Node.js 24 contract tests pass, including defaults, invalid options, forwarding and matrix artifact names. Actionlint with the repository configuration, Zizmor, formatting and Markdown checks pass. Correct the existing report fixtures to retain an available JUnit artifact when another explicit file is missing, and to use a clean ESLint file report. Hosted integration checks run on this PR.

Used by hoverkraft-tech/ci-github-common#646, hoverkraft-tech/ci-github-publish#507 and hoverkraft-tech/ci-github-container#943. One signed commit; no merge or release.

@github-actions github-actions Bot added the type: feature Introduces a new feature label Nov 10, 2026
@github-actions

github-actions Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Super-linter summary

Language Validation result
BIOME_FORMAT Pass ✅
BIOME_LINT Pass ✅
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITHUB_ACTIONS_ZIZMOR Pass ✅
GITLEAKS Pass ✅
GIT_COMMITLINT Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
JSCPD Pass ✅
MARKDOWN Pass ✅
MARKDOWN_PRETTIER Pass ✅
NATURAL_LANGUAGE Pass ✅
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅
YAML_PRETTIER Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter
Super-linter revision: 2da136927bd4a73596db63044b504547c62cb854
Super-linter version: 9.0.0

@github-actions

Copy link
Copy Markdown
Contributor

Code Coverage Report

Coverage Results

Coverage

Metric Covered Total Percentage
Lines 2 2 100.00%
Functions 1 1 100.00%

Overall: 100.00% 🟢
🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩

@codecov

codecov Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (eb4e63d) to head (e4dce7e).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #590   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            3         3           
=========================================
  Hits             3         3           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@neilime
neilime force-pushed the feat/consolidated-action-coverage branch from f5bae46 to e4dce7e Compare November 10, 2026 06:50
@neilime
neilime merged commit 9aa3f75 into main Nov 10, 2026
233 checks passed
@neilime
neilime deleted the feat/consolidated-action-coverage branch November 10, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature Introduces a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant