Feature: Better logLevel option allowing logging streams and more
See original GitHub issueFirstly just wanted to say the project is awesome! Mixed with husky
we now have some really clean hook configuration without the boilerplate we once had. 😃
Now for my question. With {"verbose": true}
set I am still unable to see eslint warnings . This can be super useful for non-blocking hints to developers. I would’ve expected to see a pretty, ahem, verbose log from all my processes when using this flag so surprised that it is missing!
I have a few proposals:
- Make
verbose: true
stream all output. - Add
loglevel
with"error"
(current),"verbose"
(current verbose)."all"
(all log streams)
Something to note might be that perhaps there is some ambiguity around “verbose logging of what lint-staged is doing” and “verbose logging of task output”.
I would be happy to work on this myself and contribute to the project! Just need a little discussion and direction in the codebase. 😃
Update
Breaking a solution into two parts:
- With verbose: true we show all output at the end of a step. This unblocks the use case of just wanting to see warnings if there was no error.
- We stream output of tasks as they are running. This unblocks the use case of wanting to see progress in longer running tasks (eg. jest). This will require more work since listr seems to take over stdout so we need to write (hopefully) a custom renderer which will allow streaming of the running task stdout.
Issue Analytics
- State:
- Created 6 years ago
- Reactions:4
- Comments:9 (4 by maintainers)
Top GitHub Comments
Sweet. Got this scoped up for my next sprint so should pick up and have a pull request Wednesday!
Might see what
concurrently
does…