Skip to content

Conversation

@keelbismark
Copy link

Checklist
  • npm install && npm run lint && npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

feat: add support for Visual Studio 2026

This pull request introduces initial support for the upcoming Visual Studio 2026 (internal version 18.0).

The key changes include:

  • Version Detection: Added '2026' and version '18.0' to the version maps in SelectVisualStudioVersion and _DetectVisualStudioVersions. This allows GYP to correctly identify VS 2026 installations.
  • Version Creation: Included a new VisualStudioVersion definition for '2026' in the _CreateVersion function.
  • Toolset Update: Set the default_toolset to v145. This aligns with the toolset version observed in the Visual Studio 2026 Insider Preview builds, resolving potential MSB8020 errors for users on the latest previews.

These changes enable developers using early versions of Visual Studio 2026 to generate and build projects with GYP seamlessly.

@cclauss
Copy link
Contributor

cclauss commented Oct 31, 2025

@StefanStojanovic, your review please. There are an increasing number of Visual Studio not found issues.

@cclauss cclauss mentioned this pull request Oct 31, 2025
@StefanStojanovic
Copy link
Contributor

@cclauss I've already approved these changes in gyp-next. It was my understanding that we should land them there and pull them to node-gyp as gyp-next update afterwards. Is this the correct flow?

@lukekarrys
Copy link
Member

@StefanStojanovic @cclauss Yeah I think that's what we agreed upon previously. Once landed and released in gyp-next there's a workflow that will automatically update them in this repo.

While on the topic: what do either of you think about a PR check that would detect changes to the gyp/ directory and fail with a message to make the changes to gyp-next instead?

@lukekarrys
Copy link
Member

Closing this in favor of #3236

@lukekarrys lukekarrys closed this Nov 7, 2025
@lukekarrys
Copy link
Member

#3236 is now merged and the next release will have support for VS2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants