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

feat(vswindows): Add WASDK workloads to match docs #148

Merged
merged 4 commits into from
Jun 20, 2023

Conversation

lukeblevins
Copy link
Contributor

Universal Windows Platform development tools represent a fairly significant dependency, which may be an obstacle to quickly getting started with Uno development.

Fortunately, the dependencies on our getting started documentation were recently updated to match the Microsoft-endorsed workloads for Windows App SDK. Accordingly, we advise new users to run the uno-check tool to pull down the required workloads if they're missing.

This PR updates our tool to no longer assume UWP workload is needed: matching the docs. It does this by using the workload ids which Microsoft details in its getting started with WASDK docs. This change is not breaking because it maintains compatibility with the previous target platform strings: "win32" and "win32desktop".

@jeromelaban
Copy link
Member

jeromelaban commented May 29, 2023

We're going to have to skip this validation in the ci validations for windows:
https://github.com/unoplatform/uno.check/actions/runs/5112431962/jobs/9190512563?pr=148#step:8:167

as it seems that the agents don't have the templates installed.

GitHub
The home for Uno.Check, a CLI tool to setup your environment to build Uno Platform apps - feat(vswindows): Add WASDK workloads to match docs · c8b1900

@lukeblevins
Copy link
Contributor Author

@jeromelaban Does this one require any additional steps from my side?

@jeromelaban
Copy link
Member

Yes, we need to skip this validation in ci steps for windows, as they don't pass, and that we won't update VS on the agent to match it.

@jeromelaban jeromelaban merged commit a87bc7e into unoplatform:main Jun 20, 2023
@lukeblevins lukeblevins deleted the dev/lubl/wasdk branch June 20, 2023 15:39
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.

3 participants