Update .NET SDK, PowerShell SDK dependencies#2168
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates pinned toolchain and dependency versions to the latest patch releases, keeping the System.Management.Automation (SMA) NuGet reference aligned with the PowerShell minimum version gate used by the PSScriptAnalyzer module loader.
Changes:
- Bump .NET SDK in
global.jsonfrom8.0.416to8.0.419. - Bump
System.Management.Automationfrom7.4.13to7.4.14. - Update
minimumPowerShellCoreVersionto7.4.14to stay consistent with the SMA version.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
global.json |
Updates the pinned .NET SDK patch version used by the repo. |
Engine/PSScriptAnalyzer.psm1 |
Keeps the PowerShell minimum supported version check aligned with the SMA package version. |
Directory.Packages.props |
Updates the centrally-managed SMA package version to 7.4.14. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Perhaps premature, the GitHub Actions images don't even have PowerShell 7.4.14 yet:
|
| { | ||
| "sdk": { | ||
| "version": "8.0.416", | ||
| "version": "8.0.419", |
There was a problem hiding this comment.
happy with it but doesn't make any difference since we use latest feature (8.0.x) anyway as per below rollForward setting
There was a problem hiding this comment.
HAHAHA yes I agree but the people who maintain internal tools that check this file don't seem to understand that.
that's ok, can do later as well. in the end it's just a reference, we don't ship SMA so whatever PowerShell patch version they are using, it's up to user to use latest patched version within 30 days after new release anyway if they want to stay supported so we don't necessarily have to enforce it too much. With 1 year release cycles, a few PS patches happen anyway |
f42292c to
b129876
Compare
Co-authored-by: andyleejordan <2226434+andyleejordan@users.noreply.github.com>
b129876 to
ad3679a
Compare
Follows the established pattern from commit
aa2180a("Update .NET SDK, PowerShell SDK, and Newtonsoft.Json").Changes
System.Management.Automation: 7.4.13 → 7.4.14 (latest 7.4.x patch)minimumPowerShellCoreVersioninEngine/PSScriptAnalyzer.psm1:'7.4.13'→'7.4.14'(kept aligned with SMA version per the in-code comment)global.json: 8.0.416 → 8.0.419Not updated (with rationale)
Newtonsoft.JsonMicrosoft.Management.InfrastructureMicrosoft.CSharpMicrosoft.Win32.RegistrySystem.Reflection.TypeExtensionsPowerShellStandard.LibraryPluralize.NET