Skip to content

Microsoft.Data.SqlClient 7.0.0#37949

Open
SimonCropp wants to merge 3 commits intodotnet:mainfrom
SimonCropp:Microsoft.Data.SqlClient-7.0.0
Open

Microsoft.Data.SqlClient 7.0.0#37949
SimonCropp wants to merge 3 commits intodotnet:mainfrom
SimonCropp:Microsoft.Data.SqlClient-7.0.0

Conversation

@SimonCropp
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings March 17, 2026 23:37
Copy link

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

Updates the centrally-managed SQL Server client dependency version used by the EF Core SQL Server provider.

Changes:

  • Bumped Microsoft.Data.SqlClient from 6.1.1 to 7.0.0 in central package management.

@SimonCropp SimonCropp marked this pull request as draft March 18, 2026 00:00
@SimonCropp SimonCropp marked this pull request as ready for review March 19, 2026 21:54
Copilot AI review requested due to automatic review settings March 19, 2026 21:54
Copy link

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

Updates the repo’s SQL Server client dependency to a newer major version, with an accompanying NuGet restore configuration change intended to unblock builds.

Changes:

  • Updated Microsoft.Data.SqlClient from 6.1.1 to 7.0.0 via central package management.
  • Added nuget.org as an additional package source in the repo NuGet.config (noted as temporary).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
NuGet.config Adds nuget.org as a package source to unblock restores.
Directory.Packages.props Bumps Microsoft.Data.SqlClient dependency to 7.0.0.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 19, 2026 21:58
Copy link

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 1 out of 1 changed files in this pull request and generated no new comments.

@roji
Copy link
Member

roji commented Mar 22, 2026

@AndriySvyryd I think we indeed want to do this for 11.0; seems like the internal nuget feed is missing packages, do you remember how to set that up?

@ErikEJ
Copy link
Contributor

ErikEJ commented Mar 22, 2026

@roji please notice the breaking change the now requires an additional package for connecting with Entra ID authentication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants