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

Cleanup of WindowsAppSDK implicit properties and project references #221

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

Arlodotexe
Copy link
Member

@Arlodotexe Arlodotexe commented Oct 25, 2024

Cleans up unnecessary explicit declaration of

  • The package reference Microsoft.Windows.SDK.BuildTools, as the same version comes from Microsoft.WindowsAppSdk via a transitive nuget reference.
  • The package reference Microsoft.Windows.CsWinRT, since the CsWinRT analyzers and .dll are inside the runtime pack for the Windows SDK.
  • The property WindowsSdkPackageVersion, since this is set automatically by the underlying SDK when running dotnet 8.0.403 or later.

Follow up to #217 (comment) and #217 (comment), closes #219.

…osoft.Windows.CsWinRT and Microsoft.Windows.SDK.BuildTools.

See #217 (comment) and #217 (comment).
@Arlodotexe
Copy link
Member Author

Arlodotexe commented Oct 25, 2024

Note that while the tooling repo has been updated to 8.0.403, the https://github.com/CommunityToolkit/Windows and https://github.com/CommunityToolkit/Labs-Windows repos are still using 8.0.201. This will need to be updated when those repos consume this tooling change.

@Arlodotexe Arlodotexe merged commit 87d5602 into main Oct 25, 2024
8 checks passed
@Arlodotexe Arlodotexe deleted the cleanup/wasdk/explicit-props-and-packagereferences branch October 25, 2024 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Remove explicit reference to Microsoft.Windows.CSWinRT
3 participants