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

Add WinAppSdk Trimming support #298

Merged
merged 4 commits into from
Sep 20, 2023
Merged

Add WinAppSdk Trimming support #298

merged 4 commits into from
Sep 20, 2023

Conversation

dansiegel
Copy link
Contributor

GitHub Issue (If applicable): closes #

PR Type

What kind of change does this PR introduce?

  • Feature

What is the current behavior?

WinAppSdk trimming is not enabled

What is the new behavior?

WinAppSdk trimming is enabled

@dansiegel dansiegel enabled auto-merge September 18, 2023 16:58
@nickrandolph
Copy link
Contributor

I think publishtrimmed should be in the publish XML files.

Copy link
Contributor

@nickrandolph nickrandolph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PublishTrimmed should either be here or in the .pubxml files. Also not sure whether there's still work to be done to ensure trimming works for Uno apps (see microsoft/CsWinRT#373)

@nickrandolph nickrandolph force-pushed the dev/ds/winappsdk-trimming branch from f550c9c to 8fc871e Compare September 20, 2023 02:54
@dansiegel dansiegel merged commit bb495ed into main Sep 20, 2023
37 checks passed
@dansiegel dansiegel deleted the dev/ds/winappsdk-trimming branch September 20, 2023 08:05
@nickrandolph
Copy link
Contributor

@mergify backport legacy/4x release/stable/4.10

@mergify
Copy link

mergify bot commented Sep 20, 2023

backport legacy/4x release/stable/4.10

✅ Backports have been created

nickrandolph added a commit that referenced this pull request Sep 20, 2023
nickrandolph added a commit that referenced this pull request Sep 20, 2023
…10/pr-298

Add WinAppSdk Trimming support (backport #298)
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.

Enable trimming in WinAppSDK templates
3 participants