Repository navigation
Update shipping dependencies for 8.0.0-preview1 - #4880
Conversation
Update all eligible direct shipping dependencies to stable minor and patch releases within their current major versions. Keep the SqlClient nuspec floors aligned with central package versions. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6e69e9bd-24d1-4058-8c3d-fa8397aec95e
There was a problem hiding this comment.
🔵 Needs a closer look
Reported shipping checks remain pending and milestone validation is failing, leaving merge readiness unverified.
No open issues 🎉
What changed in this PR
Updates shipping dependencies for 8.0.0-preview1 while preserving target frameworks, package structure, and public APIs.
Changes:
- Raises 11 central dependency versions within their existing major versions, keeping MSAL and its broker aligned.
- Synchronizes seven dependency floors across applicable SqlClient nuspec framework groups.
| File | Description |
|---|---|
| src/Microsoft.Data.SqlClient/src/Microsoft.Data.SqlClient.nuspec | Aligns packaged dependency floors with central versions. |
| Directory.Packages.props | Updates 11 shipping dependency versions. |
Review effort: Balanced
Codecov Report✅ All modified and coverable lines are covered by tests.
Additional details and impacted files@@ Coverage Diff @@
## main #4880 +/- ##
==========================================
- Coverage 72.40% 65.42% -6.98%
==========================================
Files 292 286 -6
Lines 45471 68621 +23150
==========================================
+ Hits 32924 44897 +11973
- Misses 12547 23724 +11177
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Description
Update every eligible direct shipping-package dependency to the latest listed stable minor/patch release within its current major version. All 11 selected versions are available in the configured governed feed.
Update the root central versions and synchronize the seven affected .NET dependency floors in every applicable SqlClient nuspec group, including
netstandard2.0. MSAL and its broker stay aligned; 4.91.1 also satisfies Azure.Core 1.63.0's MSAL minimum of 4.90.0.Both SNI packages, both IdentityModel packages, Buffers, Data.Common, Memory, RuntimeInformation, ValueTuple, Bcl.Cryptography, and Pkcs already use the newest stable release within their current major and remain unchanged. No eligible shipping updates were deferred.
Test-only, tooling-only, sample-only, SDK, and analyzer versions are out of scope. No public API, target framework, driver behavior, localization, product version, sibling/SNI range, asset-exclusion, or package-source changes. No transitive pinning or new direct dependencies.
Suggested release note: Update shipping NuGet dependencies to the latest compatible stable minor and servicing releases.
Issues
Fixes #4797
Testing
Validated on Windows with .NET SDK 10.0.402. No test source changes were needed: existing framework/dependency metadata tests and driver/extension tests exercise the changed dependencies.
Build and packaging passed:
The shipping build and pack completed with zero warnings/errors. Inspected all six generated nupkgs, ten framework groups, and 43 external dependency entries: all 11 updates are present; driver dependency sets, asset exclusions, and sibling/SNI ranges are preserved. Checked shipping restore graphs for the implementation, reference, unsupported-platform, Azure, and AKV projects: declared external dependencies resolve to the intended central versions, with no restore warnings/errors.
git diff --checkpassed.Runtime test results:
Commands and outcomes:
Failed at functional tests on two administrator-only certificate cases; unit tests passed. Because the aggregate stopped there, the remaining targets were run separately:
Passed.
Azure, AKV, and unit tests passed. Functional tests initially had 85 failures from the certificate fixture's child Windows PowerShell process not finding the
Cert:provider. Correcting only the process-local module path removed that fixture failure:Result: 1,330 passed, 12 skipped, and the same two administrator-only failures as net10.0. No repository or machine-wide configuration was changed.
Baseline confirmation: both remaining failures are
SqlColumnEncryptionCertificateStoreProviderShould.TestValidCertificatePaths, for the empty andmy/store paths. Both require LocalMachine certificate access unavailable to this non-elevated process. In a temporary archive of unchanged HEAD2ea42d6afa26ee17a073d5d67be664c001667809, the following commands each reproduced exactly those two failures, with the CurrentUser case passing:Metadata tests (Pester 5.9.1):
Full suite: 83 passed, 12 failed. The identical 12 failures reproduce on unchanged HEAD and involve Windows command-stub argument parsing/path handling. The relevant framework/dependency subset passes all 53 selected tests:
No update-induced failures were identified. The full local suites are not claimed green. Default test exclusions were preserved; conditional SQL/Azure cases skipped according to existing prerequisites. Live-server manual tests and Linux/macOS execution were not performed.
Guidelines
Please review the contribution guidelines before submitting a pull request:
Contributing
Code of Conduct
Best Practices
Coding Style
Review Process
Tests added or updated — no source changes; existing coverage exercised as documented above.
Public API changes documented — N/A; no public API changes.
Verified against customer repro (if applicable) — N/A; dependency maintenance task.
Ensure no breaking changes introduced — same-major stable updates; framework support and package structure preserved.