Skip to content

ci: retain native React unit failure diagnostics - #1247

Merged
blove merged 1 commit into
mainfrom
blove/native-cockpit-unit-diagnostics
Oct 4, 2026
Merged

blove merged 1 commit into
mainfrom
blove/native-cockpit-unit-diagnostics

Conversation

@blove

@blove blove commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

Native React cockpit unit failures currently reach CI as a failed Nx target without the failing assertion. Run those tests with an explicit default reporter, followed by the existing lint target, so CI retains the test name, expected value and received value while preserving failure status. Refresh only the workflow's existing inventory fingerprint.

A controlled failing fixture through the actual Nx executor exits with failure in both configurations; the explicit reporter exposes its assertion. The original Element failure on main remains unreproduced locally.

Validation: 100 workflow/matrix checks, all 984 unit tests across 19 native React examples on Node 22.14 (including all 28 Element tests), 726 React parity checks, Element lint, inventory ownership, workflow YAML and shell syntax, and diff checks passed. No application, timeout, retry, dependency or backend changes.

@vercel

vercel Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
threadplane Ready Ready Preview Oct 4, 2026 8:06am UTC

Request Review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@blove
blove merged commit d5cfeda into main Oct 4, 2026
96 checks passed

This branch was successfully deployed

1 active deployment
Preview – threadplane — 6d8ccec7 Deployed Oct 4, 2026 by vercel[bot]
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.

1 participant