Skip to content

Fix failing "Tests (windows-latest, 3.10)" CI job by pinning Windows runner to windows-2022#1057

Open
dependabot[bot] wants to merge 2 commits into
mainfrom
dependabot/npm_and_yarn/js-yaml-4.2.0
Open

Fix failing "Tests (windows-latest, 3.10)" CI job by pinning Windows runner to windows-2022#1057
dependabot[bot] wants to merge 2 commits into
mainfrom
dependabot/npm_and_yarn/js-yaml-4.2.0

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jun 22, 2026

Copy link
Copy Markdown
Contributor

The CI job "Tests (windows-latest, 3.10)" was failing because windows-latest now resolves to windows-2025, which ships with Visual Studio 2025 (major version 18). The node-gyp v11.0.0 (bundled with npm on Node 22) does not recognize VS version 18, causing the native module @vscode/windows-process-tree to fail to compile during npm ci.

Root Cause

gyp ERR! find VS unknown version "undefined" found at "C:\Program Files\Microsoft Visual Studio\18\Enterprise"
gyp ERR! find VS could not find a version of Visual Studio 2017 or newer to use

node-gyp v11 only recognizes Visual Studio up to version 17 (VS 2022). The windows-2025 runner provides VS 2025 (version 18), which node-gyp reports as "undefined" and skips.

Fix

Pinned the Windows runner from windows-latest to windows-2022 in both pr-check.yml and push-check.yml. The windows-2022 runner provides Visual Studio 2022 (version 17), which node-gyp v11 correctly identifies and uses for compiling native modules.

Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 4.1.1 to 4.2.0.
- [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md)
- [Commits](nodeca/js-yaml@4.1.1...4.2.0)

---
updated-dependencies:
- dependency-name: js-yaml
  dependency-version: 4.2.0
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code labels Jun 22, 2026
@vs-code-engineering vs-code-engineering Bot requested a review from rzhao271 June 22, 2026 18:35
rzhao271
rzhao271 previously approved these changes Jun 22, 2026
connor4312
connor4312 previously approved these changes Jun 22, 2026
Copilot AI dismissed stale reviews from connor4312 and rzhao271 via fc405a9 June 22, 2026 18:55
Copilot AI changed the title Bump js-yaml from 4.1.1 to 4.2.0 Fix failing "Tests (windows-latest, 3.10)" CI job by pinning Windows runner to windows-2022 Jun 22, 2026
Copilot AI requested a review from rzhao271 June 22, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code no-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants