Skip to content

[dotnet] shorten workload ids #12035

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

Merged
merged 2 commits into from
Jun 29, 2021

Conversation

jonathanpeppers
Copy link
Member

Context: dotnet/android#6045

We are aiming to simplify the .NET workload ids to short names like:

dotnet workload install maui
dotnet workload install android
dotnet workload install ios
dotnet workload install maccatalyst
dotnet workload install macos
dotnet workload install tvos

Updated the workload ids here, as well as the .stamp-install-workloads
make target.

Other changes:

  • Updated descriptions to match Android. Still waiting on an official
    description we should be using for each workload.

  • Fixed trailing , that showed up as JSON errors in VS Code.

Context: dotnet/android#6045

We are aiming to simplify the .NET workload ids to short names like:

    dotnet workload install maui
    dotnet workload install android
    dotnet workload install ios
    dotnet workload install maccatalyst
    dotnet workload install macos
    dotnet workload install tvos

Updated the workload ids here, as well as the `.stamp-install-workloads`
make target.

Other changes:

* Updated descriptions to match Android. Still waiting on an *official*
description we should be using for each workload.

* Fixed trailing `,` that showed up as JSON errors in VS Code.
@jonathanpeppers jonathanpeppers added the not-notes-worthy Ignore for release notes label Jun 29, 2021
@jonathanpeppers
Copy link
Member Author

jonathanpeppers commented Jun 29, 2021

FYI this is for main/preview 7 only.

/cc @GouriKumari

@vs-mobiletools-engineering-service2
Copy link
Collaborator

🔥 Tests failed catastrophically on Build (no summary found). 🔥

Result file $(TEST_SUMMARY_PATH) not found.

Pipeline on Agent
Merge 0594cbb into e2a92c1

@rolfbjarne
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vs-mobiletools-engineering-service2
Copy link
Collaborator

✅ [PR Build] Tests passed on Build. ✅

Tests passed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (no change)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

🎉 All 112 tests passed 🎉

Pipeline on Agent XAMBOT-1104.BigSur'
Merge 0594cbb into 477853a

@rolfbjarne rolfbjarne merged commit a64b030 into dotnet:main Jun 29, 2021
@jonathanpeppers jonathanpeppers deleted the dotnet-workload-install-ios branch June 29, 2021 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-notes-worthy Ignore for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants