Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -12,16 +12,18 @@ internal interface IDotNetCliInvoker

/// <summary>
/// A minimal environment for running the .NET CLI.
///
///
/// DOTNET_CLI_UI_LANGUAGE: The .NET CLI language is set to English to avoid localized output.
/// MSBUILDDISABLENODEREUSE: To ensure clean environment for each build.
/// DOTNET_SKIP_FIRST_TIME_EXPERIENCE: To skip first time experience messages.
/// DOTNET_CLI_TELEMETRY_OPTOUT: To skip any dotnet telemetry: it's unnecessary and can even cause issues.
Copy link

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

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

Doc comment wording is a bit awkward (double colon). Consider rephrasing to something like “To opt out of .NET CLI telemetry; it’s unnecessary and can cause issues.”

Suggested change
/// DOTNET_CLI_TELEMETRY_OPTOUT: To skip any dotnet telemetry: it's unnecessary and can even cause issues.
/// DOTNET_CLI_TELEMETRY_OPTOUT: Opts out of .NET CLI telemetry, which is unnecessary and can cause issues.

Copilot uses AI. Check for mistakes.
/// </summary>
static ReadOnlyDictionary<string, string> MinimalEnvironment { get; } = new(new Dictionary<string, string>
{
{"DOTNET_CLI_UI_LANGUAGE", "en"},
{"MSBUILDDISABLENODEREUSE", "1"},
{"DOTNET_SKIP_FIRST_TIME_EXPERIENCE", "true"}
{"DOTNET_SKIP_FIRST_TIME_EXPERIENCE", "true"},
{"DOTNET_CLI_TELEMETRY_OPTOUT", "1"}
});

/// <summary>
Expand Down
Loading