Skip to content

chore: upgrade all NuGet packages to latest versions - #14

Merged
Phelecks merged 3 commits into
mainfrom
copilot/upgrade-nuget-packages
Mar 23, 2026
Merged

Phelecks merged 3 commits into
mainfrom
copilot/upgrade-nuget-packages

Conversation

Copilot AI commented Mar 23, 2026 •

Copy link
Copy Markdown
Contributor

Bumps all NuGet packages to their latest versions via CPM (Directory.Packages.props), and resolves the obsolete API warnings introduced by the upgrades.

Package versions

Package Old New
Microsoft.Extensions.* (×4) 10.0.1 10.0.5
Microsoft.EntityFrameworkCore.* (×3) 10.0.1 10.0.5
Azure.Identity 1.17.1 1.19.0
Azure.Security.KeyVault.Secrets 4.8.0 4.9.0
NUnit 4.4.0 4.5.1
NUnit3TestAdapter 6.1.0 6.2.0
Microsoft.NET.Test.Sdk 18.0.1 18.3.0
Testcontainers / Testcontainers.MsSql 4.10.0 4.11.0

Obsolete API fixes

Azure.Identity 1.19.0 — ManagedIdentityCredential(string clientId, ...) is obsolete; replaced with the ManagedIdentityId-based constructors:

// before
new ManagedIdentityCredential(azureOptions.ClientId)

// after
string.IsNullOrEmpty(azureOptions.ClientId)
    ? new ManagedIdentityCredential(ManagedIdentityId.SystemAssigned)
    : new ManagedIdentityCredential(ManagedIdentityId.FromUserAssignedClientId(azureOptions.ClientId))

Testcontainers 4.11.0 — parameterless ContainerBuilder() and MsSqlBuilder() constructors are obsolete; image must now be passed directly to the constructor, and the MsSqlBuilder.MsSqlImage constant is likewise obsolete:

// before
new ContainerBuilder().WithImage(VaultImage)
new MsSqlBuilder()

// after
new ContainerBuilder(VaultImage)
new MsSqlBuilder("mcr.microsoft.com/mssql/server:2022-CU14-ubuntu-22.04")

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Upgrade all NuGet packages using CPM chore: upgrade all NuGet packages to latest versions Mar 23, 2026
Copilot AI requested a review from Phelecks March 23, 2026 14:35
@Phelecks
Phelecks marked this pull request as ready for review March 23, 2026 14:36
Copilot AI review requested due to automatic review settings March 23, 2026 14:36

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 repository’s centrally managed NuGet package versions (CPM) and applies small code changes needed to address newly introduced obsolete API warnings (Azure.Identity + Testcontainers) in the ColumnEncryptor library and its test suite.

Changes:

  • Bumped CPM versions in Directory.Packages.props (Microsoft.Extensions.*, EF Core, Azure SDK, NUnit, Testcontainers, etc.).
  • Updated Testcontainers usage in integration tests to use the new image-required constructors.
  • Updated Azure Managed Identity credential construction to use ManagedIdentityId-based APIs.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
Directory.Packages.props Central package version bumps to the latest versions.
src/ColumnEncryptor/Services/AzureKeyVaultClient.cs Replaced obsolete Managed Identity constructor usage with ManagedIdentityId-based constructors.
tests/ColumnEncryptor.Tests/HashiCorpTests.cs Updated Vault container creation to use the new ContainerBuilder(image) constructor.
tests/ColumnEncryptor.Tests/EntityFrameworkIntegrationTests.cs Updated Vault + SQL Server container builders to the new image-required constructors.

Comment thread src/ColumnEncryptor/Services/AzureKeyVaultClient.cs Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@sonarqubecloud

Copy link
Copy Markdown

@Phelecks
Phelecks merged commit 44b6409 into main Mar 23, 2026
6 checks passed
@Phelecks
Phelecks deleted the copilot/upgrade-nuget-packages branch March 23, 2026 14:43
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.

3 participants