Skip to content

[v7.1.0-preview3] Update SNI package reference to 7.1.0-preview3 - #4564

Open
priyankatiwari08 wants to merge 1 commit into
mainfrom
dev/prtiwar/sni-vbump-preview3
Open

[v7.1.0-preview3] Update SNI package reference to 7.1.0-preview3#4564
priyankatiwari08 wants to merge 1 commit into
mainfrom
dev/prtiwar/sni-vbump-preview3

Conversation

@priyankatiwari08

@priyankatiwari08 priyankatiwari08 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Description

Updates the SNI package reference to the newly published 7.1.0-preview3 build, in preparation for the MDS v7.1.0-preview3 release.

Microsoft.Data.SqlClient.SNI and Microsoft.Data.SqlClient.SNI.runtime have both been published at 7.1.0-preview3.26226.3, aligning SNI's version numbering with MDS.

Why this is required

SniVersion was last set to 6.0.2 in #4337 and has not changed since, so v7.1.0-preview1 and preview2 both shipped against SNI 6.0.2. The derived range currently evaluates to:

[6.0.2, 7.0.0)

The upper bound is exclusive at 7.0.0, so the new 7.1.0-preview3 SNI falls outside the range and would never be picked up by floating resolution. Without this explicit bump, v7.1.0-preview3 would silently ship against SNI 6.0.x.

The rest of the version alignment is already in place — following the versioning consolidation in #4336, SqlClientNextVersion is set to 7.1.0-preview3 and the entire SqlClient family resolves from it. This change brings the SNI dependency in line so that everything points at preview3.

Change

A single property update. SniVersion is declared once and fans out automatically to:

  • PackageVersion entries for Microsoft.Data.SqlClient.SNI and Microsoft.Data.SqlClient.SNI.runtime
  • The $SniVersionRange$ token substituted into Microsoft.Data.SqlClient.nuspec

The derived range now evaluates to [7.1.0-preview3.26226.3, 8.0.0). The Split('.')[0] ceiling calculation correctly yields 7 despite the prerelease suffix, and a future stable 7.1.0 will still satisfy the range since prerelease sorts below release.

Validation

  • dotnet restore resolves both packages at 7.1.0-preview3.26226.3; project.assets.json confirms the ranges as >= 7.1.0-preview3.26226.3 < 8.0.0
  • dotnet build -f net9.0 — succeeded, 0 warnings, 0 errors
  • dotnet build -f net462 — succeeded, 0 warnings, 0 errors

Checklist

  • Verified against the published SNI packages on the consumed feed
  • Restore and build succeed for both netfx and netcore target frameworks
  • No remaining references to the previous SNI version
  • Ensure no breaking changes introduced
  • Tests added or updated — n/a, dependency version change only
  • Public API changes documented — n/a, no API surface change

Release notes for v7.1.0-preview3 will follow in a separate PR.

AB#47358

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: c7425f3e-f6b8-439b-b74f-e25f8406fbf6
Copilot AI lite review requested due to automatic review settings August 21, 2026 11:49
@priyankatiwari08
priyankatiwari08 requested a review from a team as a code owner August 21, 2026 11:49
@github-project-automation github-project-automation Bot moved this to To triage in SqlClient Board Aug 21, 2026
@priyankatiwari08 priyankatiwari08 added this to the 7.1.0-preview3 milestone Aug 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 centralized SNI dependency for the v7.1.0-preview3 release.

Changes:

  • Bumps SniVersion to 7.1.0-preview3.26226.3.
  • Updates package and nuspec dependency ranges automatically.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@priyankatiwari08 priyankatiwari08 moved this from To triage to In review in SqlClient Board Aug 21, 2026
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.70%. Comparing base (ee6fabd) to head (fd45031).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4564      +/-   ##
==========================================
- Coverage   65.23%   63.70%   -1.53%     
==========================================
  Files         288      283       -5     
  Lines       44587    67648   +23061     
==========================================
+ Hits        29087    43098   +14011     
- Misses      15500    24550    +9050     
Flag Coverage Δ
CI-SqlClient ?
PR-SqlClient-Project 63.70% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

4 participants