Skip to content

Conversation

cheenamalhotra
Copy link
Member

No description provided.

@Copilot Copilot AI review requested due to automatic review settings August 15, 2025 00:28
@cheenamalhotra cheenamalhotra requested a review from a team as a code owner August 15, 2025 00:28
Copy link
Contributor

@Copilot 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

This PR updates the Azure Key Vault provider dependencies to reference Microsoft.Data.SqlClient version 6.1.1 instead of 6.1.0. The change ensures the AKV provider package is aligned with the latest SqlClient version.

Key Changes

  • Updated Microsoft.Data.SqlClient dependency from version 6.1.0 to 6.1.1 across all target frameworks
  • Updated pipeline variables to reflect the new version numbers

Reviewed Changes

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

File Description
tools/specs/add-ons/Microsoft.Data.SqlClient.AlwaysEncrypted.AzureKeyVaultProvider.nuspec Updated dependency version ranges for Microsoft.Data.SqlClient from 6.1.0 to 6.1.1 across net462, net8.0, and net9.0 target frameworks
eng/pipelines/variables/akv-official-variables.yml Updated package version variables from 6.1.0 to 6.1.1, including mdsPackageVersion and versionPatch

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Copy link

codecov bot commented Aug 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.55%. Comparing base (7a7b54e) to head (e86c51c).
⚠️ Report is 6 commits behind head on release/6.1.

Additional details and impacted files
@@               Coverage Diff               @@
##           release/6.1    #3569      +/-   ##
===============================================
- Coverage        69.69%   65.55%   -4.15%     
===============================================
  Files              281      279       -2     
  Lines            62413    61748     -665     
===============================================
- Hits             43500    40480    -3020     
- Misses           18913    21268    +2355     
Flag Coverage Δ
addons 90.82% <ø> (ø)
netcore 67.10% <ø> (-5.67%) ⬇️
netfx 69.42% <ø> (+0.17%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 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.

@paulmedynski paulmedynski merged commit 782721d into release/6.1 Aug 15, 2025
128 checks passed
@paulmedynski paulmedynski deleted the dev/cheena/akv-version-bump branch August 15, 2025 10:57
paulmedynski pushed a commit that referenced this pull request Aug 15, 2025
Updated AKV nuspec to note compatibility with MDS 6.1.1.
cheenamalhotra added a commit that referenced this pull request Aug 15, 2025
* Update AKV versions and dependencies (#3569)

Updated AKV nuspec to note compatibility with MDS 6.1.1.

* Adding historical AKV release notes (6.0.0 to 6.1.1).

* Fixed 6.0.0 notes.

---------

Co-authored-by: Cheena Malhotra <13396919+cheenamalhotra@users.noreply.github.com>
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.

2 participants