Skip to content

Remove duplicate NuGet package properties from Version.Details.props#51793

Merged
jonathanpeppers merged 2 commits intodarc-main-25319f2a-bdb3-4d70-a139-7a9acdfbdccefrom
copilot/sub-pr-51402
Nov 18, 2025
Merged

Remove duplicate NuGet package properties from Version.Details.props#51793
jonathanpeppers merged 2 commits intodarc-main-25319f2a-bdb3-4d70-a139-7a9acdfbdccefrom
copilot/sub-pr-51402

Conversation

Copy link
Contributor

Copilot AI commented Nov 18, 2025

The Version.Details.props validation was failing due to duplicate property definitions introduced during dependency flow from multiple sources.

Changes

  • Removed 18 duplicate NuGet package property definitions from the dotnet/dotnet dependencies section
  • Retained the newer versions (7.0.0-preview.2.256) from nuget/nuget.client dependencies section
  • Removed older versions (7.0.0-preview.1.2206) to resolve validation errors

The affected properties:

  • MicrosoftBuildNuGetSdkResolverPackageVersion
  • NuGetBuildTasks*PackageVersion (multiple variants)
  • NuGetCommandLine*PackageVersion
  • NuGetCommon*PackageVersion
  • All other NuGet.* package version properties

This resolves the "Missing Properties" validation error where dependencies in Version.Details.xml had conflicting property definitions in Version.Details.props.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…s.props

Co-authored-by: jonathanpeppers <840039+jonathanpeppers@users.noreply.github.com>
Copilot AI changed the title [WIP] Update source code from dotnet/dotnet Remove duplicate NuGet package properties from Version.Details.props Nov 18, 2025
Copy link
Member

@MiYanni MiYanni left a comment

Choose a reason for hiding this comment

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

This looks correct to me. The state of the file seems to indicate that it was merged from 2 different branches with changes to these properties, where one branch updated and moved them and the other likely updated them in-place. Copilot seems to be deleting the older versions of the same duplicate properties, so I think this is correct.

@jonathanpeppers jonathanpeppers marked this pull request as ready for review November 18, 2025 22:34
@jonathanpeppers jonathanpeppers merged commit 91496aa into darc-main-25319f2a-bdb3-4d70-a139-7a9acdfbdcce Nov 18, 2025
3 of 5 checks passed
@jonathanpeppers jonathanpeppers deleted the copilot/sub-pr-51402 branch November 18, 2025 22:34
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.

3 participants