Skip to content
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

Update GitHub workflow to use Windows 2022 #2348

Merged
merged 19 commits into from
Feb 16, 2022

Conversation

chenilim
Copy link
Contributor

Summary

Use windows-22 for GitHub build

@chenilim chenilim requested a review from a team as a code owner February 15, 2022 22:32
@chenilim chenilim requested review from wiggin77 and harshilsharma63 and removed request for a team February 15, 2022 22:32
@chenilim chenilim marked this pull request as draft February 15, 2022 22:35
@chenilim
Copy link
Contributor Author

Meant this to be a draft. I updated .NET to 4.8 (from 4.7) to see if that fixes the Windows 2022 dependencies.

@chenilim chenilim changed the title Workflow windows 2022 Update GitHub workflow to use Windows 2022 Feb 15, 2022
@chenilim
Copy link
Contributor Author

Ok, after updating the Windows 10 SDK path, this passes the GitHub build with the Windows Server 2022 image. The new requirements are: .NET 4.8 and Windows 10 SDK 10.0.19041.0 (added to readme).

@chenilim chenilim marked this pull request as ready for review February 16, 2022 01:59
@chenilim chenilim requested a review from sbishel February 16, 2022 01:59
@harshilsharma63
Copy link
Member

@chenilim does this affect the OS/system requirements for running the Boards desktop app?

@chenilim
Copy link
Contributor Author

@harshilsharma63, yes .NET 4.8 is now required (up from 4.7.2). Documentation. However, the Windows SDK itself doesn't affect the version of Windows required (as long as new APIs aren't used).

README.md Outdated Show resolved Hide resolved
@chenilim chenilim merged commit 5ba81b4 into mattermost-community:main Feb 16, 2022
@chenilim chenilim deleted the workflow-windows-2022 branch February 16, 2022 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants