Running task steps per folder fails if containing folder is empty.

[dave-cross]

I set up a similar task to the one described here: https://github.com/gulpjs/gulp/blob/master/docs/recipes/running-task-steps-per-folder.md

Instead of running JS or SCSS, it runs a sprite generator.

The issue is that if there're no folders returned from getFolders(), the task (and any sequenced/parallel tasks), fail. Without subfolders in the src path, .map won't run... which means gulp scripts don't fire... leaving us with a null stream.

Is there any way to return an "empty" stream if no folders are present or inform gulp that the task has run successfully in the same instance?

[yocontra]

I think it would be pretty obvious that you should put something in the folder you're processing, but I added a check for it in the example code in the commit above. You can just return if there are no folders to merge.

[phated]

@contra thats not correct and you didn't follow the conventional changelog style

[yocontra]

@phated What isn't correct about it? Did the way synchronous functions work change in undertaker?

[phated]

Sync functions are not supported. That's been in forever.

[dave-cross]

@contra, That conditional didn't work. I still get errors that the task didn't complete. As for 'putting something in the folder I'm processing", I did mention that it's a sprite generator. It is possible SVGs haven't been made and added yet.

Thank you both for answering.

[yocontra]

@CloudSociety Replace the return with return done() and add a done arg to your task function. Published a fix for it - I forgot master = 4.0 docs now, not the 3.x stuff. If you're using 3.x the return will work fine.

[phated]

I've rebased and cleaned up the result.