Skip to content

Conversation

SimenB
Copy link
Contributor

@SimenB SimenB commented May 11, 2016

Fixes #47

I'm unsure why this is needed, may be because of the wrapping that gulp does? Seems like the same issue as pgherveou/gulp-awspublish#13, so I just copied the solution

@coveralls
Copy link

coveralls commented May 11, 2016

Coverage Status

Coverage remained the same at 84.507% when pulling 352fe78 on SimenB:patch-1 into 1350804 on lazd:master.

@lazd
Copy link
Owner

lazd commented May 11, 2016

@SimenB sounds like we needed this, should I merge and release?

@SimenB
Copy link
Contributor Author

SimenB commented May 11, 2016

Yeah, otherwise the end event isn't emitted, for some reason.

@SimenB
Copy link
Contributor Author

SimenB commented Jun 3, 2016

@lazd ping :)

@lazd lazd merged commit e96c934 into lazd:master Jun 3, 2016
@SimenB SimenB deleted the patch-1 branch June 10, 2016 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants