Skip to content

Catch node failures in ci#446

Open
mcbarton wants to merge 1 commit intocompiler-research:mainfrom
mcbarton:catch-node-failures
Open

Catch node failures in ci#446
mcbarton wants to merge 1 commit intocompiler-research:mainfrom
mcbarton:catch-node-failures

Conversation

@mcbarton
Copy link
Collaborator

When we was experimenting with reducing the initial memory when creating the xeus-cpp tests, the node tests failed to run, but the ci didn't catch it and fail (see https://github.com/compiler-research/xeus-cpp/actions/runs/21941897688/job/63369501220#step:5:182). This PR will attempt to catch these failures, and once they are caught the cmake change will be reverted.

@mcbarton
Copy link
Collaborator Author

@mcbarton mcbarton force-pushed the catch-node-failures branch 2 times, most recently from 2667d86 to 02b11ff Compare February 17, 2026 10:09
@mcbarton mcbarton marked this pull request as ready for review February 17, 2026 10:09
@mcbarton mcbarton force-pushed the catch-node-failures branch from 5d89886 to eb217ca Compare February 17, 2026 10:13
@mcbarton mcbarton force-pushed the catch-node-failures branch from 90d742f to 4256df6 Compare February 17, 2026 10:50
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.94%. Comparing base (d5dfb61) to head (4256df6).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #446   +/-   ##
=======================================
  Coverage   81.94%   81.94%           
=======================================
  Files          21       21           
  Lines         853      853           
  Branches       87       87           
=======================================
  Hits          699      699           
  Misses        154      154           
🚀 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.

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