Skip to content

ci: add no-wasm-simd option#4533

Merged
mcollina merged 5 commits into
mainfrom
no-wasm-simd
Sep 20, 2025
Merged

ci: add no-wasm-simd option#4533
mcollina merged 5 commits into
mainfrom
no-wasm-simd

Conversation

@Uzlopak

@Uzlopak Uzlopak commented Sep 9, 2025

Copy link
Copy Markdown
Contributor

This relates to...

Rationale

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

@Uzlopak Uzlopak changed the title No wasm simd ci: add no-wasm-simd option Sep 9, 2025
@mcollina

Copy link
Copy Markdown
Member

This conflicts now

@Uzlopak Uzlopak requested a review from Copilot September 10, 2025 23:11

Copilot AI 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.

Pull Request Overview

This PR adds a new CI configuration option to disable WASM SIMD functionality during testing. It introduces a no-wasm-simd input parameter to the Node.js workflow and creates a new test job that runs with WASM SIMD disabled.

  • Adds no-wasm-simd input parameter to the Node.js workflow
  • Creates a new CI job specifically for testing with WASM SIMD disabled
  • Sets the UNDICI_NO_WASM_SIMD environment variable across all test steps

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/nodejs.yml Adds no-wasm-simd input parameter and sets UNDICI_NO_WASM_SIMD environment variable in all test steps
.github/workflows/ci.yml Creates new test-with-no-wasm-simd job and includes it in the required checks

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/workflows/ci.yml
@Uzlopak Uzlopak requested a review from Copilot September 11, 2025 07:49

Copilot AI 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.

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina merged commit 8761011 into main Sep 20, 2025
34 of 35 checks passed
@Uzlopak Uzlopak deleted the no-wasm-simd branch September 20, 2025 20:46
@github-actions github-actions Bot mentioned this pull request Jan 5, 2026
slagiewka pushed a commit to slagiewka/undici that referenced this pull request Feb 14, 2026
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.

3 participants