-
Notifications
You must be signed in to change notification settings - Fork 327
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
[WAM] Adding more dev apps to test msalruntime existence #4044
Conversation
@@ -28,6 +28,14 @@ steps: | |||
custom: 'workload' | |||
arguments: 'restore .\src\client\Microsoft.Identity.Client\Microsoft.Identity.Client.csproj' | |||
|
|||
- task: Bash@3 | |||
displayName: Install MAUI |
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.
This task adds ~6 minutes to the build time. Wondering if we can move these WAM devapps in to a separate sln, which we build and run these tests on?
It's ok for now to improve the tests. Just trying to be mindful each time we update the build pipeline to make sure we don't get into situation again when we have to refactor the pipeline since our builds take long time again.
dotnet workload install maui --source https://aka.ms/dotnet6/nuget/index.json --source https://api.nuget.org/v3/index.json | ||
dotnet workload install android ios maccatalyst tvos macos maui wasm-tools --source https://aka.ms/dotnet6/nuget/index.json --source https://api.nuget.org/v3/index.json |
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.
nit: we don't need to install all these workloads.
Adding more dev apps to test msalruntime existence
this PR adds the following dev apps
runtimeidentifiers