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

Timing output should include both the core task and the total time #23

Open
ScottWeinstein opened this issue Jan 26, 2014 · 2 comments
Open

Comments

@ScottWeinstein
Copy link

So this suggestion was originally made on gulpjs here (gulpjs/gulp#189)

If we have the following

gulp.task("compile", ["lint", "sass"], function() {
     // do something which takes 50ms
});

where lint and sass are async

in the current version the output of this might be

lint: 200 ms
sass: 250 ms
compile: 50 ms

which is quite misleading. Really the time it took for compile to complete is Max(lintTime, sassTime) + compileTime

Helpful output for any task would be in include inclusive and exclusive time, say

lint: 200 ms
sass: 250 ms
compile: 50 ms / 350 ms

@robrich
Copy link
Owner

robrich commented Mar 24, 2014

Seems a duplicate of #14

@iandanforth
Copy link

👍

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

3 participants