-
Notifications
You must be signed in to change notification settings - Fork 20
[FSSDK-11731] windows version update #382
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FSSDK-11731] windows version update #382
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates Windows runner versions across GitHub workflows to comply with IT standards, specifically updating from windows-2019
and windows-latest
to windows-2022
. The changes include adding .NET Framework reference assembly installation steps to maintain compatibility with older framework versions when using the newer Windows runner.
- Updated all Windows runners to use
windows-2022
- Added manual installation of .NET Framework 4.0 and 4.5 reference assemblies
- Removed the SonarQube workflow entirely
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
File | Description |
---|---|
.github/workflows/sonarqube.yml | Complete removal of SonarQube workflow |
.github/workflows/csharp_release.yml | Updated runners to windows-2022 and added .NET Framework reference assembly installation |
.github/workflows/csharp.yml | Updated runner to windows-2022 and added .NET Framework reference assembly installation with error tolerance |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with the YAML configs and the removal of SonarQube. The resulting runs of in GitHub Actions look good.
The Copilot suggestions are still something to consider, but given the successful executions, I can't argue with holding this merge.
Summary
Windows version update to comply with IT standards
Test plan
Existing test should pass
Issues