Skip to content

fix(ci): pin Windows desktop build runner to windows-2022#317

Closed
skevetter wants to merge 1 commit into
mainfrom
0b89-d4e9-fix-windows-runner
Closed

fix(ci): pin Windows desktop build runner to windows-2022#317
skevetter wants to merge 1 commit into
mainfrom
0b89-d4e9-fix-windows-runner

Conversation

@skevetter

Copy link
Copy Markdown
Contributor

Summary

  • Pin Windows Build Desktop App runner from windows-latest to windows-2022 to fix node-gyp/electron-rebuild failure caused by VS 2026 (v18) on the updated windows-latest image
  • Remove the configure Windows build tools step (vswhere/GYP_MSVS_OVERRIDE_PATH workaround from PR fix(ci): resolve 3 desktop build failures in release workflow #314) since pinning the runner makes it unnecessary

windows-latest now ships VS 2026 (v18) which node-gyp doesn't
recognize, breaking electron-rebuild for node-pty. Pin to windows-2022
(VS 2022 / v17) and remove the vswhere GYP_MSVS_OVERRIDE_PATH
workaround that is no longer needed.
@netlify

netlify Bot commented May 17, 2026

Copy link
Copy Markdown

Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit 17402cd
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6a0a43973394d30008e4a44b

@coderabbitai

coderabbitai Bot commented May 17, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e8b7066f-6fdb-4068-8a4d-3bb66b82ecbe

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@skevetter

Copy link
Copy Markdown
Contributor Author

Closing as duplicate — superseded by PR #319 which covers the same fix (pin windows-2022). PR #317 was from a previous SM team.

@skevetter skevetter closed this May 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant