Repository navigation
Update dependency packages - #2072
Open
alerickson wants to merge 2 commits into
Open
alerickson wants to merge 2 commits into
alerickson wants to merge 2 commits into
Conversation
alerickson
requested review from
Sydney Smith (SydneyhSmith),
Aditya Patwardhan (adityapatwardhan),
Anam Navied (anamnavi) and
shammu1
as code owners
October 22, 2026 19:22
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Member
Author
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
🔵 Needs a closer look
Major dependency upgrades and assembly packaging changes need build and packaged-module validation across supported PowerShell runtimes.
No open issues 🎉
What changed in this PR
Updates PSResourceGet’s dependencies and packaged assemblies, while preserving publishing support for registered HTTP repositories.
Changes:
- Upgrades NuGet libraries, Azure.Identity, System.Text.Json, and cryptography dependencies.
- Updates the assemblies included in the module package.
- Explicitly allows HTTP connections in the NuGet publish call.
| File | Description |
|---|---|
src/code/PublishHelper.cs |
Preserves HTTP publishing support with the updated NuGet API. |
src/code/Microsoft.PowerShell.PSResourceGet.csproj |
Updates dependency versions. |
doBuild.ps1 |
Adds required dependency assemblies and removes System.ValueTuple from packaging. |
Review effort: Balanced
Member
Author
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
Member
Author
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
This branch has not been deployed
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.
PR Summary
Update dependencies to the latest stable releases:
Adapt publishing to NuGet 7’s required
allowInsecureConnections parameter, preserving existing support for registered HTTP repositories.Allow newer bundled dependencies to satisfy older version requests. After the package update,
Publish-PSResourceto a container registry failed with:Azure.CoreandSystem.Memory.Dataare built againstSystem.Text.Json10.0.0.3, but the module now ships 10.0.0.13. The module's assembly resolver only accepted an exact match on the full assembly name, including the version, so it rejected the newer bundled DLL. PowerShell has no binding redirects for module dependencies, so the load failed. The same mismatch affected other upgraded dependencies, such as Microsoft.Bcl.AsyncInterfaces and Microsoft.Extensions.*.With the changes in ModuleInitAndCleanup.cs, bundled dependencies are now indexed by simple name instead of full name. A requested assembly is resolved from the bundle when the bundled copy has the same name, the same or a higher version, the same culture, the same public key token. Requests for versions newer than the bundled copy, or with a different public key token, are still rejected. The requesting assembly must still be this module or one of its bundled dependencies, matched exactly as before.
PR Context
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.