-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
Maybe provide some fallback if not on a TTY #11
Comments
Without a tty, colors and backtracking / replacing existing output is impossible. So perhaps the output just needs to drop all animations and colors? Perhaps:
E.g.
|
Or just go verbose.
If I am hunting down a problem in a Jenkins / Travis build, missing log data I suspect might be relevant is going to be a lot more frustrating than it is locally, where it is easy to rerun. |
I guess ideally you could trigger "plain text output without control chars" and "verbose output" independently. So it might be something like
or (if verbose)
|
PS: But yes, verbose mode alone would also work for me right now 😉 |
Should be possible to activate with |
listr-verbose-renderer almost does the job. Just need a way to show all output if necessary (SamVerschueren/listr-verbose-renderer#1) and implement auto-switching to verbose renderer when in a CI scenario, piping to a file or similar. |
Moving the discusion to #33. PR was added to add this. |
Hey!
For https://github.com/andywer/npm-launch I was thinking about a plain-text fallback in case
!require('tty').isatty(process.stdout)
. Right now you get all those control characters in the output when piping to a file or similar. In my particular case I have Jenkins/Travis builds in mind.But maybe you are interested to solve that edge case directly in Listr?
The text was updated successfully, but these errors were encountered: