Skip to content

Fix #18086: suppress NuGet restore stdout under FSI --quiet#19808

Open
T-Gro wants to merge 3 commits into
mainfrom
fix/issue-18086
Open

Fix #18086: suppress NuGet restore stdout under FSI --quiet#19808
T-Gro wants to merge 3 commits into
mainfrom
fix/issue-18086

Conversation

@T-Gro
Copy link
Copy Markdown
Member

@T-Gro T-Gro commented May 26, 2026

Summary

When --quiet ( cConfigB.noFeedback) is active, route the captured stdout of the NuGet restore subprocess to stderr instead of stdout so that warnings like NU1608 and MSBuild Determining projects to restore... chatter no longer pollute FSI stdout.

Default (non-quiet) behavior is unchanged.
esult.StdError continues to go to stderr unconditionally.

Changes

  • src/Compiler/Interactive/fsi.fs – When
    oFeedback is set, redirect restore stdout to stderr instead of stdout.
  • ** ests/.../FsiCliTests.fs** – Add a test verifying that --quiet suppresses NuGet restore stdout.
  • Release notes – Added entry under Fixed for 11.0.100.

Fixes #18086

T-Gro and others added 3 commits May 26, 2026 08:46
When --quiet (tcConfigB.noFeedback) is active, route the captured stdout of the NuGet restore subprocess to stderr instead of stdout so that warnings like NU1608 and MSBuild `Determining projects to restore...` chatter no longer pollute FSI stdout. Default (non-quiet) behavior is unchanged. result.StdError continues to go to stderr unconditionally.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

❗ Release notes required


✅ Found changes and release notes in following paths:

Warning

No PR link found in some release notes, please consider adding it.

Change path Release notes path Description
src/Compiler docs/release-notes/.FSharp.Compiler.Service/11.0.100.md No current pull request URL (#19808) found, please consider adding it

@github-actions github-actions Bot added the AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed label May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

Nuget warnings in fsi are emitted to stdout and cannot be suppressed with --quiet

1 participant