-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Update WindowsAppSDK to 1.8 #32174
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
base: net10.0
Are you sure you want to change the base?
Update WindowsAppSDK to 1.8 #32174
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 the Windows App SDK from version 1.7 to 1.8 and modernizes Windows platform support across multiple projects. The update addresses issue #30858 by ensuring consistent Windows SDK versions and proper package references.
Key changes:
- Updated Windows App SDK to version 1.8.251003001 and Windows SDK Build Tools to 10.0.26100.4654
- Added
Microsoft.WindowsAppSDKpackage reference to graphics and essentials projects for consistent Windows support - Fixed asset path handling in Windows integration tests to accommodate both RID-graph and non-RID-graph scenarios
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| eng/Versions.props | Updated Windows App SDK and Build Tools package versions to latest releases |
| src/Essentials/src/Essentials.csproj | Added Microsoft.WindowsAppSDK package reference for Windows platform support |
| src/Graphics/src/Graphics.Skia/Graphics.Skia.csproj | Added Microsoft.WindowsAppSDK package reference for Windows builds |
| src/Graphics/src/Graphics.Win2D/Graphics.Win2D.csproj | Added Microsoft.WindowsAppSDK package reference for consistency |
| src/Graphics/src/Graphics/Graphics.csproj | Added duplicate Microsoft.WindowsAppSDK package reference |
| src/Controls/src/Xaml/Controls.Xaml.csproj | Removed Windows-specific DefineConstants and PRI generation properties |
| Directory.Build.targets | Added empty TypeScript targets to prevent unwanted output handling during publish |
| src/TestUtils/src/Microsoft.Maui.IntegrationTests/WindowsTemplateTest.cs | Modified RID path construction to handle both RID-graph and non-RID-graph scenarios |
src/TestUtils/src/Microsoft.Maui.IntegrationTests/WindowsTemplateTest.cs
Show resolved
Hide resolved
56262f0 to
0609452
Compare
This pull request updates several project files to improve Windows platform support and modernize dependencies. The main changes involve updating Windows SDK package versions, adding the
Microsoft.WindowsAppSDKpackage to relevant projects, and making adjustments for publishing and output handling.Fixes #30858
Attempt 1 #30665
Attempt 2 #31281
Dependency updates:
MicrosoftWindowsAppSDKPackageVersionto1.8.251003001andMicrosoftWindowsSDKBuildToolsPackageVersionto10.0.26100.4654ineng/Versions.props, ensuring projects use the latest Windows SDKs.Project file improvements for Windows support:
Microsoft.WindowsAppSDKas aPackageReferencetoEssentials.csproj,Graphics.Skia.csproj,Graphics.Win2D.csproj, and duplicated inGraphics.csprojto ensure consistent Windows App SDK usage across graphics and essentials projects. [1] [2] [3] [4]Controls.Xaml.csproj, simplifying project configuration for Windows builds.Build and publishing process adjustments:
GetTypeScriptOutputForPublishing,GetTypeScriptCopyToOutputDirectoryItems) inDirectory.Build.targetsto improve TypeScript output handling during publishing.WindowsTemplateTest.csto better handle runtime identifier (RID) directories when publishing packaged Windows apps.