Skip to content

fix(ci): pin desktop Windows build to windows-2022 for node-gyp compat#316

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

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

Conversation

@skevetter

Copy link
Copy Markdown
Contributor

Summary

  • Pin desktop Windows build runner from windows-latest to windows-2022 to fix node-gyp/electron-rebuild failure
  • windows-latest now resolves to win25-vs2026 with VS 18 which @electron/node-gyp doesn't support (only up to VS 17/2022)
  • Simplify configure step: remove stale vswhere/GYP_MSVS_OVERRIDE_PATH logic, just set GYP_MSVS_VERSION=2022
  • CLI build retains windows-latest (Go builds don't use node-gyp)

@netlify

netlify Bot commented May 17, 2026

Copy link
Copy Markdown

Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit ed774a6
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6a0a432580151900081c36a0

@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: 40ff4c03-79bd-48bd-8243-b3a5c361b66c

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 in favor of PR #317 which is further along in CI.

@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