PowerShell / PSScriptAnalyzer

Download ScriptAnalyzer from PowerShellGallery
https://www.powershellgallery.com/packages/PSScriptAnalyzer/
MIT License
1.85k stars 373 forks source link

Upgrade to use .NET 6 since PowerShell 7.0 is now out out of support #1873

Closed bergmeister closed 1 year ago

bergmeister commented 1 year ago

PR Summary

Only PowerShell 7.2 and 7.3 are supported at the moment, which are using .NET 6 and 7, therefore we can bump our version of .NET as well.

fixes #1919

PR Checklist

fflaten commented 1 year ago

Remember to bump devcontainer-image as well :)

peterhoeg commented 1 year ago

Doesn't this need to change too: https://github.com/PowerShell/PSScriptAnalyzer/blob/9f456d05b95421090fa63e1bb7378231291b36b6/PSCompatibilityCollector/Microsoft.PowerShell.CrossCompatibility/Microsoft.PowerShell.CrossCompatibility.csproj#L5

?

bergmeister commented 1 year ago

Doesn't this need to change too:

https://github.com/PowerShell/PSScriptAnalyzer/blob/9f456d05b95421090fa63e1bb7378231291b36b6/PSCompatibilityCollector/Microsoft.PowerShell.CrossCompatibility/Microsoft.PowerShell.CrossCompatibility.csproj#L5

?

No, net 462 is for windows powershell and .net 6 still implements .net standard 2