Skip to content

ci(release): pin windows build to windows-2022#294

Merged
jcanizalez merged 1 commit intomainfrom
ci/pin-windows-2022
May 7, 2026
Merged

ci(release): pin windows build to windows-2022#294
jcanizalez merged 1 commit intomainfrom
ci/pin-windows-2022

Conversation

@jcanizalez
Copy link
Copy Markdown
Collaborator

Summary

  • windows-latest is mid-transition to windows-2025-vs2026 (per the runner-image deprecation notice) and is currently missing a Visual Studio install
  • The v0.2.1-beta.1 release build failed at electron-builder installAppDeps when node-gyp tried to rebuild node-pty: `Could not find any Visual Studio installation to use`
  • Pin the windows job to windows-2022 (stable VS2022) until the new image stabilises

Test plan

  • Re-tag v0.2.1-beta.1 after merge and confirm windows .exe artifacts upload

windows-latest is mid-transition to windows-2025-vs2026 and is currently
missing Visual Studio, breaking node-pty's node-gyp rebuild during
electron-builder installAppDeps. Pin to windows-2022 (stable VS2022)
until the new image stabilises.
Copilot AI review requested due to automatic review settings May 7, 2026 18:10
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Pins the Windows release build job to a stable GitHub Actions runner image to avoid breakages during the current windows-latest image transition (notably missing Visual Studio for node-gyp/node-pty rebuilds during Electron packaging).

Changes:

  • Change the Windows release job runner from windows-latest to windows-2022 to ensure VS2022 is available for native module builds.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jcanizalez jcanizalez merged commit f689bfa into main May 7, 2026
8 checks passed
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.

2 participants