Address feedback from peer review #50567
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
adegeo
reviewed
Dec 15, 2025
| The .NET SDK displays text similar to the following when you first run one of the [.NET CLI commands](index.md) (for example, `dotnet build`). Text might vary slightly depending on the version of the SDK you're running. This "first run" experience is how Microsoft notifies you about data collection. | ||
|
|
||
| > [!NOTE] | ||
| > **Breaking change:** The behavior of telemetry messages written to `stderr` has changed in recent versions of the .NET SDK. For more information, see [dotnet CLI commands log non-command-relevant data to stderr](../compatibility/sdk/10.0/dotnet-cli-stderr-output.md). |
Contributor
There was a problem hiding this comment.
Do you think it's good to remove this considering this is new behavior introduced in .NET 10?
Contributor
Author
There was a problem hiding this comment.
I moved this note down 10 lines. The recommendation was to move the note so that the output isn't separated from its intro text.
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request addresses peer review feedback for the .NET SDK telemetry documentation by improving organization, clarity, and privacy guidance. The changes reorganize telemetry data collection information into a more scannable tabbed format by SDK version, enhance privacy disclosures, and improve formatting consistency throughout the document.
Key changes:
- Reorganized telemetry data collection from a table format to version-specific tabs with bulleted lists for better readability
- Enhanced privacy guidance with earlier placement, clearer language, and a link to the Microsoft Privacy Statement
- Improved formatting consistency by using code formatting for environment variable names in tables
Co-authored-by: Copilot <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This pull request updates the .NET SDK telemetry documentation to improve clarity, privacy guidance, and formatting. The most important changes include a new section on avoiding inadvertent disclosure of sensitive information, a more readable and organized list of collected telemetry data by SDK version, and improved formatting of environment variable and agent tables.
Fixes #49866
Internal previews