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

Telemetry PR link out of date #5697

Closed
svick opened this issue May 30, 2018 · 1 comment
Closed

Telemetry PR link out of date #5697

svick opened this issue May 30, 2018 · 1 comment
Assignees
Labels
⌚ Not Triaged Not triaged

Comments

@svick
Copy link
Contributor

svick commented May 30, 2018

This article links to dotnet/cli#2145, which seems to be the initial PR that added telemetry to .Net Core 1.0. Since then, how telemetry is collected was changed, which means that the link provides inaccurate image of how exactly is telemetry collected by .Net Core SDK.

Is there a better link that could be included in the article? If not, maybe the link should be removed?


Document Details

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

@mairaw
Copy link
Contributor

mairaw commented May 31, 2018

I think we could remove this link although the desire could be that customers would see the code. Maybe there's a better place to link to now like the folder where the code lives. @richlander any thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⌚ Not Triaged Not triaged
Projects
None yet
Development

No branches or pull requests

5 participants