-
Notifications
You must be signed in to change notification settings - Fork 21
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
cargo::Builder::quiet
is too coarse
#1014
Comments
smoelius
added a commit
that referenced
this issue
Jan 17, 2024
Bandaid until we come up with a better API for #1014.
smoelius
added a commit
that referenced
this issue
Jan 17, 2024
Bandaid until we come up with a better API for #1014.
smoelius
added a commit
to smoelius/dylint
that referenced
this issue
Jan 19, 2024
smoelius
added a commit
to smoelius/dylint
that referenced
this issue
Jan 19, 2024
smoelius
added a commit
to smoelius/dylint
that referenced
this issue
Jan 19, 2024
smoelius
added a commit
to smoelius/dylint
that referenced
this issue
Jan 20, 2024
smoelius
added a commit
to smoelius/dylint
that referenced
this issue
Jan 20, 2024
smoelius
added a commit
to smoelius/dylint
that referenced
this issue
Jan 21, 2024
Fixed by #1025 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
quiet
prevents "header" messages:dylint/internal/src/cargo.rs
Lines 107 to 109 in 1ffeac5
quiet
also prevents constructed commands from writing to stderr:dylint/internal/src/cargo.rs
Lines 140 to 142 in 1ffeac5
In dylint/src/metadata/cli/mod.rs, we don't use
quiet
because we want allow writing to stderr:dylint/dylint/src/metadata/cli/mod.rs
Lines 245 to 246 in 1ffeac5
But this then creates unwanted "header" messages.
The text was updated successfully, but these errors were encountered: