Skip to content

Fix NU1107 message guidance for CPVM with transitive pinning#7235

Open
Copilot wants to merge 7 commits intodevfrom
copilot/fix-nu1107-guidance-error
Open

Fix NU1107 message guidance for CPVM with transitive pinning#7235
Copilot wants to merge 7 commits intodevfrom
copilot/fix-nu1107-guidance-error

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 25, 2026

Bug

Fixes: NuGet/Home#12277

Description

When CPVM (Central Package Version Management) with transitive pinning is enabled, the NU1107 error message previously told users to "Install/reference X directly to project Y" — guidance that doesn't apply with transitive pinning.
They can just "pin" that package.

PR Checklist

  • Meaningful title, helpful description and a linked NuGet/Home issue
  • Added tests
  • Link to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc.

When a version conflict involves a centrally managed transitive package,
provide guidance to update the centrally managed package version instead
of suggesting to install/reference the package directly.

New message: "Version conflict detected for {0} transitively pinned
centrally managed package in project {1}. Update the centrally managed
package version to a higher version."

Co-authored-by: nkolev92 <2878341+nkolev92@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NuGet/NuGet.Client/sessions/42973248-ebc6-4845-bdaa-c3f070e5ac66
Copilot AI changed the title [WIP] Fix NU1107 message guidance for CPVM with transitive pinning Fix NU1107 message guidance for CPVM with transitive pinning Mar 25, 2026
Copilot AI requested a review from nkolev92 March 25, 2026 21:10
@nkolev92
Copy link
Copy Markdown
Member

@copilot Update the PR description with the PR template.

You may need to push a dummy change to get your routines to actually update the body.

Agent-Logs-Url: https://github.com/NuGet/NuGet.Client/sessions/df082b6e-9a66-4079-9c51-24f4b07d696c

Co-authored-by: nkolev92 <2878341+nkolev92@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 26, 2026

@copilot Update the PR description with the PR template.

You may need to push a dummy change to get your routines to actually update the body.

Updated the PR description to follow the PR template in commit d3c65c0. It now includes the Fixes: link to NuGet/Home#12277, a description of changes, and the PR checklist.

@nkolev92
Copy link
Copy Markdown
Member

This is more complex than I thought.

@nkolev92 nkolev92 closed this Mar 27, 2026
@nkolev92 nkolev92 reopened this Mar 27, 2026
@nkolev92
Copy link
Copy Markdown
Member

I tweaked the implementation.

This was a requirements challenge.

@nkolev92 nkolev92 marked this pull request as ready for review March 27, 2026 16:14
@nkolev92 nkolev92 requested a review from a team as a code owner March 27, 2026 16:14
@nkolev92 nkolev92 requested a review from donnie-msft March 27, 2026 16:14
@nkolev92 nkolev92 requested a review from donnie-msft March 27, 2026 19:07
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.

NU1107 message provides poor guidance when CPVM + transitive pinning is enabled

3 participants