Repository navigation
chore: upgrade all NuGet packages to latest versions - #14
Merged
Merged
Conversation
Co-authored-by: Phelecks <47243363+Phelecks@users.noreply.github.com> Agent-Logs-Url: https://github.com/Tricksfor-Organization/ColumnEncryptor/sessions/ee17f9a9-1378-4cde-bd1e-55d28086a193
Copilot
AI
changed the title
[WIP] Upgrade all NuGet packages using CPM
chore: upgrade all NuGet packages to latest versions
Mar 23, 2026
Phelecks
marked this pull request as ready for review
March 23, 2026 14:36
Contributor
There was a problem hiding this comment.
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. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Phelecks
approved these changes
Mar 23, 2026
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.



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
Microsoft.Extensions.*(×4)Microsoft.EntityFrameworkCore.*(×3)Azure.IdentityAzure.Security.KeyVault.SecretsNUnitNUnit3TestAdapterMicrosoft.NET.Test.SdkTestcontainers/Testcontainers.MsSqlObsolete API fixes
Azure.Identity1.19.0 —ManagedIdentityCredential(string clientId, ...)is obsolete; replaced with theManagedIdentityId-based constructors:Testcontainers4.11.0 — parameterlessContainerBuilder()andMsSqlBuilder()constructors are obsolete; image must now be passed directly to the constructor, and theMsSqlBuilder.MsSqlImageconstant is likewise obsolete:💡 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.