[v7.1.0-preview2] Release Notes and vBump#4422
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR prepares the 7.1.0-preview2 release by updating the default SqlClient version, bumping pipeline version variables, and adding/advertising the preview2 release notes in both the release-notes index and the top-level changelog.
Changes:
- Bumped
SqlClientVersionDefaultto7.1.0-preview2. - Added release notes for
7.1.0-preview2and updated the 7.1 release-notes index + top-levelCHANGELOG.md. - Updated selected OneBranch / CI variables to emit
7.1.0-preview2package versions.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Microsoft.Data.SqlClient/Versions.props | Updates the default SqlClient version to preview2. |
| release-notes/7.1/README.md | Adds preview2 to the 7.1 release-notes index table. |
| release-notes/7.1/7.1.0-preview2.md | Introduces the preview2 release notes content. |
| eng/pipelines/onebranch/variables/package-variables.yml | Bumps the OneBranch SqlClient preview package version string. |
| eng/pipelines/libraries/ci-build-variables.yml | Bumps the CI mdsPackageVersion prerelease identifier to preview2. |
| CHANGELOG.md | Adds a new top-level changelog entry for 7.1.0-preview2. |
priyankatiwari08
force-pushed
the
dev/prtiwar/vbump-release-notes
branch
from
781d858 to
e8e4b0b
Compare
priyankatiwari08
commented
Jul 3, 2026
paulmedynski
requested changes
Jul 3, 2026
github-project-automation
Bot
moved this from To triage
to Waiting for customer
in SqlClient Board
priyankatiwari08
force-pushed
the
dev/prtiwar/vbump-release-notes
branch
from
e8e4b0b to
aa15530
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4422 +/- ##
==========================================
- Coverage 65.78% 63.87% -1.92%
==========================================
Files 286 282 -4
Lines 43696 66658 +22962
==========================================
+ Hits 28745 42577 +13832
- Misses 14951 24081 +9130
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:
|
priyankatiwari08
force-pushed
the
dev/prtiwar/vbump-release-notes
branch
from
aa15530 to
56d78bf
Compare
* vBump in Versions.props, OneBranch pipelines, and CI/PR pipelines to v7.1.0-preview2 * Release notes for v7.1.0-preview2
priyankatiwari08
force-pushed
the
dev/prtiwar/vbump-release-notes
branch
from
56d78bf to
06bf786
Compare
… note Address Paul's PR feedback: the phrase 'unchanged from 7.0.2' attached to both FileVersion and AssemblyVersion, making it sound like FileVersion was also unchanged. Split into two sentences so only AssemblyVersion is described as unchanged from 7.0.2. Applied across all 4 per-package 7.1.0-preview2 release-notes files (AzureKeyVaultProvider, Extensions.Azure, Extensions.Abstractions, Internal.Logging).
paulmedynski
reviewed
Jul 6, 2026
Address review feedback on PR #4422: - Rewrite Extensions.Azure preview2 notes to document WAM broker support properly (removes false "no functional changes" claim and wrong "added in 7.0.2" pointer). WAM was added in 7.1.0-preview2 per PR #4288's milestone, not 7.0.2. - Remove WAM Broker Support section from core preview2 notes; WAM lives only in the Extensions.Azure package. - Drop Microsoft.Identity.Client / Microsoft.Identity.Client.Broker dependency-bump bullet from core preview2 notes and CHANGELOG; MSAL packages are referenced only by Extensions.Azure, not by core Microsoft.Data.SqlClient (which uses the Wilson IdentityModel stack). Keep the .NET 10 SDK bump (PR #4287) in the core notes. - Split CHANGELOG "Re-shipped" bullet so Extensions.Azure gets its own callout for the WAM addition; AKV, Extensions.Abstractions, and Internal.Logging remain grouped as alignment-only.
paulmedynski
previously approved these changes
Jul 6, 2026
mdaigle
reviewed
Jul 6, 2026
apoorvdeshmukh
previously approved these changes
Jul 6, 2026
apoorvdeshmukh
left a comment
apoorvdeshmukh
left a comment
Contributor
There was a problem hiding this comment.
Reviewed the vBump changes. Looks good.
Approving to get the vBump changes merged.
mdaigle
requested changes
Jul 6, 2026
- Drop 'pool v2' framing from Configurable Idle Connection Timeout section; frame the new keyword/property as applying to the existing connection pool as well (channel pool changes will be announced in preview 3). - Add Switch.Microsoft.Data.SqlClient.UseOverallConnectTimeoutForPoolWait AppContext switch details (default false, opt-in to true) to the Connection Timeout Now Propagates Through the Pool section, and reframe Who Benefits/Impact to lead with unchanged default behavior.
priyankatiwari08
dismissed stale reviews from apoorvdeshmukh and paulmedynski
via
d222a7a
paulmedynski
previously approved these changes
Jul 6, 2026
mdaigle
previously requested changes
Jul 6, 2026
…notes Per Malcolm's follow-up review, channel pool (pool v2) changes will be announced in preview 3. Removes: - The 'Automatic Pool Pruning and Shutdown for the Channel-Based Pool' section (What Changed / Who Benefits / Impact) from release-notes/7.1/7.1.0-preview2.md. - The matching 'Implemented pool shutdown for ChannelDbConnectionPool and added automatic pool size reduction (pruning) for the channel-based pool' bullet from CHANGELOG.md's Changed section. PRs #4302 and #4304 are no longer referenced in the preview2 notes.
paulmedynski
approved these changes
Jul 7, 2026
paulmedynski
dismissed
mdaigle’s stale review
I reviewed on Malcolm's behalf and changes meet his feedback.
apoorvdeshmukh
approved these changes
Jul 7, 2026
github-project-automation
Bot
moved this from Waiting for customer
to Done
in SqlClient Board
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Versions.props, OneBranch pipelines, and CI/PR pipelines to v7.1.0-preview2CHANGELOG.mdentry and updatedrelease-notes/7.1/README.mdindex