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

Log only the "Finished" messages and not the "Starting" ones #177

Closed
marcofugaro opened this issue Jan 3, 2019 · 3 comments · Fixed by #260
Closed

Log only the "Finished" messages and not the "Starting" ones #177

marcofugaro opened this issue Jan 3, 2019 · 3 comments · Fixed by #260
Projects
Milestone

Comments

@marcofugaro
Copy link

Hey, it would be really useful if I could tell Gulp to log only when a task is done, not when it starts.
I don't actually care when the task starts during development. I only care about how much time it took and if it ran successfully.

I've tried the log-level option and it just turns the starting/finished logs on or off.

What do you think?

@sttk
Copy link
Contributor

sttk commented Jan 5, 2019

@marcofugaro Now we consider that gulp-cli enable to customize colors and messages of logs. If it is achieved, gulp-cli will be able to output an empty line too instead of starting log.

@phated phated added this to the 3.0 milestone Apr 20, 2019
@phated
Copy link
Member

phated commented May 20, 2020

@sttk I think we should avoid logging any line that is "falsey" so they can set the theming to false to disable.

@sttk
Copy link
Contributor

sttk commented May 20, 2020

@phated To do that, we should modify lib/shared/log/log-levels.js in PR #161 as follows:

  • L33: log.on(level, console.error);log.on(function(s) { if (s) console.error(s): });
  • L35: log.on(level, console.log);log.on(function(s) { if (s) console.log(s): });

@phated phated added this to To do in v5 Oct 21, 2020
v5 automation moved this from To do to Done Mar 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v5
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants