Skip to content

Conversation

Copy link

Copilot AI commented Sep 5, 2025

Resolves a NuGet package version conflict that was preventing successful dotnet restore operations on the test projects.

Problem

The restore operation was failing with NU1107 errors due to conflicting version requirements for Microsoft.CodeAnalysis.CSharp.Workspaces:

error NU1107: Version conflict detected for Microsoft.CodeAnalysis.CSharp.Workspaces. 
Install/reference Microsoft.CodeAnalysis.CSharp.Workspaces 4.14.0 directly to project 
Pomelo.EntityFrameworkCore.MySql.Tests to resolve this issue.

Pomelo.EntityFrameworkCore.MySql.Tests -> Microsoft.EntityFrameworkCore.Design 10.0.0-preview.7.25380.108 
-> Microsoft.CodeAnalysis.CSharp.Workspaces (>= 4.14.0)

Pomelo.EntityFrameworkCore.MySql.Tests -> Microsoft.CodeAnalysis 4.10.0 
-> Microsoft.CodeAnalysis.CSharp.Workspaces (= 4.10.0)

Root Cause

  • Microsoft.EntityFrameworkCore.Design version 10.0.0-preview.7.25380.108 requires Microsoft.CodeAnalysis.CSharp.Workspaces >= 4.14.0
  • The project was pinning all Microsoft.CodeAnalysis.* packages to version 4.10.0 in Directory.Packages.props
  • This created an irreconcilable version conflict

Solution

Updated all Microsoft.CodeAnalysis packages in Directory.Packages.props from version 4.10.0 to 4.14.0:

  • Microsoft.CodeAnalysis: 4.10.0 → 4.14.0
  • Microsoft.CodeAnalysis.CSharp: 4.10.0 → 4.14.0
  • Microsoft.CodeAnalysis.CSharp.Features: 4.10.0 → 4.14.0
  • Microsoft.CodeAnalysis.CSharp.Workspaces: 4.10.0 → 4.14.0
  • Microsoft.CodeAnalysis.VisualBasic.Features: 4.10.0 → 4.14.0

This change maintains consistency across all CodeAnalysis packages while satisfying the minimum version requirement from EF Core Design.

Fixes #82.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI changed the title [WIP] Fix dependency issue Fix Microsoft.CodeAnalysis dependency version conflict Sep 5, 2025
Copilot AI requested a review from renemadsen September 5, 2025 13:45
@renemadsen renemadsen marked this pull request as ready for review September 5, 2025 13:50
@renemadsen renemadsen merged commit 180338a into master Sep 5, 2025
0 of 54 checks passed
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.

Fix dependency issue

2 participants