Skip to content

Conversation

@Mikejo5000
Copy link
Contributor

Title

added ms.workload values for reporting

Summary

Added metadata attribute "ms.workload" with values of "dotnet" and "dotnetcore"

Fixes #Issue_Number

NA

Details

This is to provide easier reporting on docs related to VS workloads.

Suggested Reviewers

@mairaw

@mairaw
Copy link
Contributor

mairaw commented Dec 21, 2017

@Mikejo5000 this looks good for the .NET Core workload. But what does the dotnet workload value represents? Those topics under the standard folder apply for both Core and Desktop.

@mairaw
Copy link
Contributor

mairaw commented Dec 21, 2017

Is that field multi-value? The idea would be that we manually enter the core values to the standard folder later?

@Mikejo5000
Copy link
Contributor Author

@mairaw The field is multi-value. My first thought for the \standard folder was to just use "dotnet" and add "dotnetcore" when verified on .NET Core. I'm rethinking that now; it is probably best to add "dotnetcore" to the \standard topics now, and remove that value later if it is determined that .NET Core is not supported. That will likely be much more accurate, correct? I'll update the PR here.

@mairaw
Copy link
Contributor

mairaw commented Dec 22, 2017

That seems better. Thanks @Mikejo5000!

@mairaw mairaw merged commit ae572ab into dotnet:master Dec 22, 2017
@Mikejo5000
Copy link
Contributor Author

@mairaw Thanks, Maira!

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.

2 participants