docs icon indicating copy to clipboard operation
docs copied to clipboard

Update .NET SDK and .NET CLI telemetry about collecting data for mobile and WebAssembly workloads

Open ivanpovazan opened this issue 1 year ago • 5 comments

In .NET8, with https://github.com/dotnet/runtime/pull/90208 we introduced telemetry for collecting data about mobile and WebAssembly workloads.

This change should also be documented in https://github.com/dotnet/docs/blob/e234edcaae55e45e93fbb4931a7ae70f16789f6d/docs/core/tools/telemetry.md by updating adequate tables in Data points section.


cc: @steveisok @vitek-karas @baronfel


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.


Associated WorkItem - 251070

ivanpovazan avatar Jan 26 '24 14:01 ivanpovazan

I am not sure about setting the right labels, so please feel free to select the correct ones.

ivanpovazan avatar Jan 26 '24 14:01 ivanpovazan

Hey @tdykstra you addressed me to this issue.

To be honest, I'm not exactly sure how to do this correctly. But, I tried my best with https://github.com/dotnet/docs/compare/main...timdeschryver:issue/39276?expand=1. I wanted to ask you if I can create a PR for these changes because I'm not entirely confident if these changes are what's expected for this issue.

I also suppose the additional property added in https://github.com/dotnet/runtime/pull/90950 needs to be documented.

timdeschryver avatar May 02 '24 18:05 timdeschryver

@ivanpovazan @steveisok I'm not sure how to translate the code in https://github.com/dotnet/runtime/pull/90208 and https://github.com/dotnet/runtime/pull/90950 to the table in the doc. @timdeschryver made a proposed update that you can see here. Is this correct? If not, can you please update a couple of data points in the table to show us how to interpret the code? And likewise if any data points from https://github.com/dotnet/runtime/pull/90208 were missed in the proposed table changes.

tdykstra avatar May 02 '24 19:05 tdykstra

Hello, I would suggest opening a PR so it would be easier to review the changes and comment/align with what has been introduced in: https://github.com/dotnet/runtime/pull/90208 and https://github.com/dotnet/runtime/pull/90950

ivanpovazan avatar May 03 '24 08:05 ivanpovazan

Sounds good! I opened #40746 to track the changes, feedback is welcome.

timdeschryver avatar May 03 '24 17:05 timdeschryver

@timdeschryver Thanks again for taking this on!

tdykstra avatar May 31 '24 20:05 tdykstra