Skip to content
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

Test for TTY Context before printing progress #420

Closed
melloc01 opened this issue Apr 30, 2018 · 2 comments
Closed

Test for TTY Context before printing progress #420

melloc01 opened this issue Apr 30, 2018 · 2 comments

Comments

@melloc01
Copy link

melloc01 commented Apr 30, 2018

Do you want to request a feature or report a bug?
Improvement

What is the current behavior?
Outputs progress on non-TTY contexts, generating loads of ouput

If the current behavior is a bug, please provide the steps to reproduce.

  1. webpack --progress in a non-TTY context

What is the expected behavior?
To not print progress. There's a node api for TTY: https://nodejs.org/api/tty.html

If this is a feature request, what is motivation or use case for changing the behavior?
CI output gets bloated with process output.

@melloc01 melloc01 changed the title Test for TTY Context before printing Test for TTY Context before printing progress Apr 30, 2018
@evenstensberg
Copy link
Member

Have you tried setting env variables for supports-color? https://github.com/chalk/supports-color/blob/master/test.js#L156-L160

@evenstensberg evenstensberg changed the title Test for TTY Context before printing progress Question: Test for TTY Context before printing progress May 18, 2018
@evenstensberg
Copy link
Member

Fixed at #452

@melloc01 melloc01 changed the title Question: Test for TTY Context before printing progress Test for TTY Context before printing progress May 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants