Skip to content

[dotnet-trace] Remove preview banner from collect-linux - #5970

Open
mdh1418 wants to merge 1 commit into
dotnet:mainfrom
mdh1418:dotnet_trace_collect_linux_remove_preview
Open

[dotnet-trace] Remove preview banner from collect-linux#5970
mdh1418 wants to merge 1 commit into
dotnet:mainfrom
mdh1418:dotnet_trace_collect_linux_remove_preview

Conversation

@mdh1418

@mdh1418 mdh1418 commented Aug 20, 2026

Copy link
Copy Markdown
Member

No description provided.

@mdh1418
mdh1418 requested review from hoyosjs and noahfalk and a lite review from Copilot August 20, 2026 18:21
@mdh1418
mdh1418 requested a review from a team as a code owner August 20, 2026 18:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the “preview feature” banner that was printed when running dotnet-trace collect-linux, and updates functional tests to reflect the new console output.

Changes:

  • Removed the preview banner Console.WriteLine(...) block from CollectLinux.
  • Updated CollectLinuxCommandFunctionalTests expected output helpers to no longer include the preview banner lines.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/Tools/dotnet-trace/CommandLine/Commands/CollectLinuxCommand.cs Removes the preview banner printed at the start of collect-linux.
src/tests/dotnet-trace/CollectLinuxCommandFunctionalTests.cs Updates expected output and helper methods to match the new (banner-free) output.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@noahfalk

Copy link
Copy Markdown
Member

Is there a corresponding PR to change the docs?

@mdh1418

mdh1418 commented Aug 21, 2026

Copy link
Copy Markdown
Member Author

Yes, just put it up here dotnet/docs#55628

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.

4 participants