Skip to content

Conversation

@mcollina
Copy link
Member

Temporarily disable the Node.js compiled --shared-builtin-undici/undici-path CI workflow to address testing issues.

🤖 Generated with Claude Code

This relates to...

Rationale

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

Temporarily disable the Node.js compiled --shared-builtin-undici/undici-path CI workflow to address testing issues.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Matteo Collina <hello@matteocollina.com>
@mcollina mcollina requested review from richardlau and ronag June 20, 2025 15:21
@richardlau
Copy link
Member

Is anyone going to be looking at the failures? Since the tests pass on Node.js 20, couldn't we keep that and disable the tests for 22/24?

@metcoder95
Copy link
Member

I've started to look at them, but haven't had more time to spend yet

@Uzlopak Uzlopak merged commit b8f7fc2 into main Jun 27, 2025
66 of 70 checks passed
@Uzlopak Uzlopak deleted the disable-shared-builtin-ci branch June 27, 2025 15:14
@richardlau
Copy link
Member

What I've been able to figure out re. the failures: nodejs/node#58865

@github-actions github-actions bot mentioned this pull request Jul 18, 2025
@richardlau
Copy link
Member

Fixes have landed in upstream Node.js that allow this CI to pass again. I've opened #4426 to re-enable it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants