Skip to content

Add NuGet dependency version bounds for EF Core inter-package references#38114

Merged
AndriySvyryd merged 8 commits intomainfrom
copilot/add-upper-limits-nuget-packages
Apr 16, 2026
Merged

Add NuGet dependency version bounds for EF Core inter-package references#38114
AndriySvyryd merged 8 commits intomainfrom
copilot/add-upper-limits-nuget-packages

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 15, 2026

This will prevent obscure exception messages caused by mixing packages from different major versions

Copilot AI and others added 6 commits April 15, 2026 17:54
… same major version

Agent-Logs-Url: https://github.com/dotnet/efcore/sessions/ff2d320a-c5bc-4702-9474-118456fa3f9e

Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
…ackageDependencyUpperBound property

Agent-Logs-Url: https://github.com/dotnet/efcore/sessions/13466f2d-2d65-4c9c-b3d0-747244ca9f81

Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
Agent-Logs-Url: https://github.com/dotnet/efcore/sessions/13466f2d-2d65-4c9c-b3d0-747244ca9f81

Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
…/Versions.props

Agent-Logs-Url: https://github.com/dotnet/efcore/sessions/f4e8a257-37b6-4ed4-b4b9-7948be16d57e

Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
…ces and central UseExplicitPackageVersions target

Agent-Logs-Url: https://github.com/dotnet/efcore/sessions/16f92744-3ba6-41e4-a2ff-d21d9b3ed385

Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
…mport order issue

Agent-Logs-Url: https://github.com/dotnet/efcore/sessions/c6c7a825-392c-4345-a8d1-5c09b849fd0f

Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
Comment thread src/EFCore.Cosmos/EFCore.Cosmos.csproj Outdated
Comment thread Directory.Build.targets Outdated
…tReference items

Agent-Logs-Url: https://github.com/dotnet/efcore/sessions/60788937-58f7-4241-9b1c-6aba77ab6481

Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
Copy link
Copy Markdown

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

Centralizes and standardizes NuGet dependency version ranges for EF Core inter-package references by moving per-project logic into a single build target and using a shared upper-bound version property.

Changes:

  • Adds a computed NextMajorVersion property in eng/Versions.props for dependency upper bounds.
  • Annotates inter-repo ProjectReference items in packable projects with PackageVersion ranges.
  • Introduces a central UseExplicitPackageVersions target to flow PackageVersion metadata into nuspec dependency generation.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/Microsoft.Data.Sqlite/Microsoft.Data.Sqlite.csproj Adds PackageVersion range to the ProjectReference.
src/EFCore/EFCore.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.Sqlite/EFCore.Sqlite.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.Sqlite.NTS/EFCore.Sqlite.NTS.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.Sqlite.Core/EFCore.Sqlite.Core.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.SqlServer/EFCore.SqlServer.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.SqlServer.NTS/EFCore.SqlServer.NTS.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.SqlServer.HierarchyId/EFCore.SqlServer.HierarchyId.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.Relational/EFCore.Relational.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.Proxies/EFCore.Proxies.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.InMemory/EFCore.InMemory.csproj Adds PackageVersion ranges to internal ProjectReference items.
src/EFCore.Design/EFCore.Design.csproj Adds PackageVersion range to the ProjectReference.
src/EFCore.Cosmos/EFCore.Cosmos.csproj Adds PackageVersion ranges to internal ProjectReference items.
eng/Versions.props Introduces computed NextMajorVersion property for version range upper bounds.
Directory.Build.targets Adds central UseExplicitPackageVersions target to propagate explicit ranges into nuspec generation.

Comment thread Directory.Build.targets Outdated
@AndriySvyryd AndriySvyryd changed the title Centralize NuGet dependency version bounds for EF Core inter-package references Add NuGet dependency version bounds for EF Core inter-package references Apr 15, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@AndriySvyryd AndriySvyryd marked this pull request as ready for review April 15, 2026 20:54
@AndriySvyryd AndriySvyryd requested a review from a team as a code owner April 15, 2026 20:54
Copilot AI review requested due to automatic review settings April 15, 2026 20:54
@AndriySvyryd AndriySvyryd assigned roji and unassigned AndriySvyryd and Copilot Apr 15, 2026
Copy link
Copy Markdown

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

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

Copy link
Copy Markdown
Member

@roji roji left a comment

Choose a reason for hiding this comment

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

Makes a lot of sense, I've done this in EFCore.PG too.

@roji roji assigned AndriySvyryd and unassigned roji Apr 16, 2026
@AndriySvyryd AndriySvyryd merged commit f874cae into main Apr 16, 2026
17 of 19 checks passed
@AndriySvyryd AndriySvyryd deleted the copilot/add-upper-limits-nuget-packages branch April 16, 2026 06:42
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.

4 participants