search

OverZealous / run-sequence

Run a series of dependent gulp tasks in order
MIT License
962 stars 61 forks source link

Better error reporting #69

Closed VojtaStanek closed 8 years ago

VojtaStanek commented 8 years ago

fixes #65

OverZealous commented 8 years ago

Does this render name of the task in which the error occurred?

VojtaStanek commented 8 years ago

No, it doesn't. Do you think it would be better to use failed package's name as a name for the error?

OverZealous commented 8 years ago

I just want to make sure that it's easy to debug, since the originating task can get lost in asynchronous errors. That's why the original error included e.task.

Maybe simply change the first argument to 'run-sequence(' + e.task + ')'. That should at least help in debugging the error.

VojtaStanek commented 8 years ago

Agree! Changed!

OverZealous commented 8 years ago

Thanks!

I assume you meant to replace the line, rather than duplicate it? ;-)

Fix that, and I'll get this merged in and published!

VojtaStanek commented 8 years ago

Oh, yes. ;)

Story: Why did I duplicated it? Because I edited it, but atom removed some whitespaces from empty lines, so I had to stage&commit it in GUI (GitKraken), where I staged the adding, but forgot to stage the removal. ;)

OverZealous commented 8 years ago

Awesome, thanks!

OverZealous commented 8 years ago

The update is published as v1.2.0