gulp-rsync icon indicating copy to clipboard operation
gulp-rsync copied to clipboard

return needed

Open carltonj2000 opened this issue 8 years ago • 3 comments

Your first usage example has a return but the second one does not. That made all the difference for me. Without the return the rsync would not work. So maybe you can update your second usage example so that it has a return.

carltonj2000 avatar Nov 17 '16 18:11 carltonj2000

G'day,

That's a good point. We will modify it ASAP. Cheers for that.

Eki

On 18 Nov 2016 5:25 AM, "Carlton Joseph" [email protected] wrote:

Your first usage example has a return but the second one does not. That made all the difference for me. Without the return the rsync would not work. So maybe you can update your second usage example so that it has a return.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jerrysu/gulp-rsync/issues/44, or mute the thread https://github.com/notifications/unsubscribe-auth/AAaTwRUB5MPkdZy-hW3Z-80WfR5vyUgMks5q_JwUgaJpZM4K1pLO .

ebaskoro avatar Nov 19 '16 00:11 ebaskoro

Huge bump. Had no idea this return stated was needed in gulp.

c-johnson avatar Mar 20 '18 01:03 c-johnson

Can anyone explain why this is? I get how not returning properly can mess up Gulp's default task ordering, but I don't understand why this wouldn't actually execute the rsync function.

c-johnson avatar Mar 20 '18 19:03 c-johnson