Skip to content

Fix dotnet/sdk#49921 by making the dgspec inner build silent #49927

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

Merged
merged 1 commit into from
Jul 25, 2025

Conversation

baronfel
Copy link
Member

The inner build creating the dgspec that the package add command relied on needs to set for silent running, as do other 'inner' builds that happen in the CLI.

@Copilot Copilot AI review requested due to automatic review settings July 24, 2025 16:34
@baronfel baronfel requested a review from a team as a code owner July 24, 2025 16:34
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes issue #49921 by making the inner dgspec build silent to prevent unnecessary output clutter. The change adds the -v:quiet verbosity flag to the MSBuild arguments used when creating the dependency graph specification file during package add operations.

  • Adds quiet verbosity flag to reduce output noise from inner builds
  • Includes explanatory comment describing the purpose of the change

@baronfel baronfel requested a review from a team July 24, 2025 16:34
@baronfel baronfel added Area-NuGet Area-CLI cli-ux Issues and PRs that deal with the UX of the CLI (exit codes, log output, verbs/options, and so on) labels Jul 24, 2025
@baronfel baronfel merged commit d4e9bbc into dotnet:main Jul 25, 2025
31 checks passed
@baronfel baronfel deleted the package-add-silent-inner-build branch July 25, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-CLI Area-NuGet cli-ux Issues and PRs that deal with the UX of the CLI (exit codes, log output, verbs/options, and so on)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants