Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@akoeplinger
Copy link
Member

@akoeplinger akoeplinger commented Aug 13, 2025

So the VMR can properly override the version used.

Microsoft Reviewers: Open in CodeFlow

So the VMR can properly override the version used.
Copilot AI review requested due to automatic review settings August 13, 2025 17:28
@akoeplinger akoeplinger requested a review from a team as a code owner August 13, 2025 17:28
Copy link
Contributor

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

This PR moves the System.Private.Windows.Core.TestUtilities dependency from a hardcoded version in Versions.props to the centralized dependency management system, allowing the VMR (Virtual Monorepo) to properly override the version used.

Key changes:

  • Removes hardcoded version definition from Versions.props
  • Adds dependency entry to Version.Details.xml with proper metadata
  • Adds corresponding version properties to Version.Details.props

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
eng/Versions.props Removes hardcoded SystemPrivateWindowsCoreTestUtilitiesVersion property
eng/Version.Details.xml Adds System.Private.Windows.Core.TestUtilities dependency with version and source metadata
eng/Version.Details.props Adds package version property and maps it to the version variable

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@dotnet-policy-service dotnet-policy-service bot added the PR metadata: Label to tag PRs, to facilitate with triage label Aug 13, 2025
@akoeplinger
Copy link
Member Author

cc @harshit7962

Copy link
Member

@harshit7962 harshit7962 left a comment

Choose a reason for hiding this comment

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

LGTM

@harshit7962 harshit7962 merged commit e103fb8 into dotnet:main Aug 14, 2025
8 checks passed
@akoeplinger akoeplinger deleted the fix-testutilities branch August 14, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR metadata: Label to tag PRs, to facilitate with triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants