Replies: 2 comments
-
Thanks for the question. This is actually handled within the progress bar library we use, indicatif. I've added a comment at bbf84b3. Are you seeing an issue in practice related to this? |
Beta Was this translation helpful? Give feedback.
0 replies
-
Closing since this has been answered. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
ReporterStderr
's documentation points out that not all terminals will be PTYs:nextest/nextest-runner/src/reporter/imp.rs
Lines 22 to 26 in e1578c7
However, in the displayer, the bar is only disabled if an option (and/or two) is passed, or if Nextest detects that it's running in CI -- else, it'll show the progress bar:
nextest/nextest-runner/src/reporter/displayer/imp.rs
Lines 75 to 98 in e1578c7
Am I missing where the piped case is handled in
ReporterStderrImpl::TerminalWithBar
, or is this a regression?Beta Was this translation helpful? Give feedback.
All reactions