-
Notifications
You must be signed in to change notification settings - Fork 50
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
return needed #44
Comments
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:
|
Huge bump. Had no idea this return stated was needed in gulp. |
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. |
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.
The text was updated successfully, but these errors were encountered: