Skip to content

lookup: skip win32-arm64 for serialport - #1221

Open
MikeMcC399 wants to merge 1 commit into
nodejs:mainfrom
MikeMcC399:skip/serialport-win32-arm64
Open

MikeMcC399 wants to merge 1 commit into
nodejs:mainfrom
MikeMcC399:skip/serialport-win32-arm64

Conversation

@MikeMcC399

Copy link
Copy Markdown
Contributor

Situation

serialport fails in citgm-smoker running under win11-vs2022_clang-arm64 in Jenkins. The error logs contain:

error MSB6006: "llvm-lib.exe" exited with code 1.
...
 npm error gyp ERR! node-gyp -v v9.4.1
 npm error gyp ERR! not ok

Documentation on https://serialport.io/docs/next/guide-platform-support does not list support for Windows arm64 and the package is not set up to support this platform.

Change

Add a skip entry win32-arm64 into lookup.json for serialport to avoid testing on this failing and unsupported operating system / architecture platform combination.

Checklist
  • npm test passes
  • contribution guidelines followed
    here

Signed-off-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>
@codecov-commenter

codecov-commenter commented Oct 28, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.11%. Comparing base (98ba854) to head (be0c243).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1221   +/-   ##
=======================================
  Coverage   96.11%   96.11%           
=======================================
  Files          29       29           
  Lines        2213     2213           
=======================================
  Hits         2127     2127           
  Misses         86       86           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

@MikeMcC399
MikeMcC399 marked this pull request as ready for review October 28, 2026 09:04
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.

serialport fails in win11-vs2022_clang-arm64

2 participants