Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 14, 2025

Ports the defensive null-safety improvements from PR #16313 (release/9.0) to the release/8.0 branch.

Changes

The LocateVisualStudio function in eng/common/tools.ps1 now guards against missing global.json properties:

  • Check $GlobalJson.tools.vs existence before assignment
  • Validate $vsRequirements is non-null before accessing .version and .components
  • Add -ErrorAction SilentlyContinue to Get-Member calls

Prevents errors when global.json lacks a vs node in tools section, enabling VS 17.14+ with preview SDK configurations.

To double check:

Original prompt

Port change #16313 to target the release/8.0 branch as well


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: marcpopMSFT <12663534+marcpopMSFT@users.noreply.github.com>
Copilot AI changed the title [WIP] Port changes from PR 16313 to release/8.0 branch Port PR #16313 vsRequirements refactor to release/8.0 Nov 14, 2025
Copilot finished work on behalf of marcpopMSFT November 14, 2025 21:59
Copilot AI requested a review from marcpopMSFT November 14, 2025 21:59
@marcpopMSFT
Copy link
Member

chcosta is going to try to bake prerelease support configuration for VS into the images so hopefully we don't need this change.

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