GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||
GeezerVison is showing github.com — Upgrade build to .NET 10 and thus raise minimum PS7 version to 7.6 by Copilot · Pull Request #2266 · PowerShell/PSScriptAnalyzer Conversation
|
There was an error while loading. Please reload this page.
| File | Description |
|---|---|
| README.md | Updates SDK and recommended Visual Studio requirements. |
| global.json | Pins the build SDK to 10.0.401. |
| .devcontainer/Dockerfile | Switches the container base image to .NET 10. |
| .devcontainer/devcontainer.json | Updates the container display name to .NET 10. |
Review effort: Balanced
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Co-authored-by: bergmeister <[email protected]>
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Windows CI is failing because Microsoft.WindowsDesktop.App.Ref lacks an exact nuget.org mapping, causing version 8.0.33 to restore from the Azure feed and fail with HTTP 401.
Review effort: Balanced
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
No active review issues are recorded for the .NET 10 build and development environment update.
No open issues 🎉
Review effort: Balanced
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Copilot upgrading .NET from 8 to 10 also means upgrading the minimum supported version of PowerShell 7 version from 7.4 to 6.6 as we have tests for that in our manifest script (.psdm1 file). Please update it and other places that referenced PowerShell 7.4 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Co-authored-by: bergmeister <[email protected]>
Updated the module loader minimum to PowerShell 7.6.6 in b0be65a. I kept the SMA 7.4.14 reference because SMA 7.6.6 targets only net10, while this project retains its net8 target; the net8 Engine build and focused tests pass. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Copilot why not powershell 7.6.9? it's latest powershell version and by the time psscriptanalyzer releases it would've been out for long enough as guidance is to update powershell within a month after release of patch. alos, I think we should update SMA reference and update everyting that targets net8 to net10 because net10 is implicitly the minomum .NET version due to the minimumm PowerShell 7 upgrade to 7.6 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Co-authored-by: bergmeister <[email protected]>
Updated to PowerShell/SMA 7.6.9 and retargeted the Engine and Rules projects plus the PowerShell 7 build selection to .NET 10.0, while retaining the net462 and netstandard2.0 targets. Changes are in 4b51b00; the full solution builds in Debug and Release. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Co-authored-by: bergmeister <[email protected]>
|
Copilot it seems CI is still using PowerShell 7.6.6 so please lower minimum version from 7.6.9 to 7.6.6 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Co-authored-by: bergmeister <[email protected]>
Lowered the loader minimum to PowerShell 7.6.6 in commit |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
No active review issues are blocking approval of the .NET 10 migration.
No open issues 🎉
Review effort: Balanced
Sorry, something went wrong.
There was an error while loading. Please reload this page.
At least 1 approving review is required to merge this pull request.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
Update the build to use .NET 10
There was an error while loading. Please reload this page.