Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
AArnott opened this issue Nov 29, 2022 · 0 comments
Labels
Area:RestoreCPM Central package management Functionality:Restore Priority:2 Issues for the current backlog. SeQuality Type:Bug

Comments

@AArnott
Copy link
Contributor

AArnott commented Nov 29, 2022

NuGet Product Used

dotnet.exe

Product Version

7.0.100

Worked before?

No response

Impact

It's more difficult to complete my work

Repro Steps & Context

After applying transitive pinning to the VS-Platform repo, I got this error:

error NU1107: Version conflict detected for Microsoft.CodeAnalysis.Common. Install/reference Microsoft.CodeAnalysis.Common 4.3.0-1.22228.4 directly to project Microsoft.VisualStudio.Extensibility.EditorHostService.UnitTests to resolve this issue.

This error gives irrelevant guidance, IMO. Adding a PackageReference to the conflicted package is no longer the appropriate step. Given we have CPVM with transitive pinning, the only fix is to add a PackageVersion item for the conflicted package to Directory.Packages.props.

Verbose Logs

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area:RestoreCPM Central package management Functionality:Restore Priority:2 Issues for the current backlog. SeQuality Type:Bug
Projects
None yet
Development

No branches or pull requests

3 participants