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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants