Skip to content
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

Emitting kill to stream does not kill livereload #95

Open
chiel opened this issue Oct 13, 2015 · 4 comments
Open

Emitting kill to stream does not kill livereload #95

chiel opened this issue Oct 13, 2015 · 4 comments

Comments

@chiel
Copy link

chiel commented Oct 13, 2015

Hello,

I'm using gulp-webserver to run end to end tests with gulp-protractor. Once the tests have been run I'd like to be able to shut the server down. However, it would appear that emitting kill to the stream does not kill off the livereload server.

The reason livereload is even enabled is because the server is also used during regular development. Of course I can use a quick fix and only run livereload when the server is not used in conjunction with the tests, but figured desired behaviour would be to kill livereload as well.

Cheers.

@liabru
Copy link

liabru commented Nov 8, 2015

+1

@yoaquim
Copy link

yoaquim commented May 27, 2016

+1, having same issue. trying to kill gulp-webserver once karma's done, but emitting the kill event does nothing.

@DSoko2
Copy link

DSoko2 commented Nov 16, 2017

+1 In addition to the explanations of @chiel I figured out, that the exact issue is that the change detection via "watch" is not stopped on shutdown. The livereload server however is closed correctly. I'm going to propose a fix

@DSoko2
Copy link

DSoko2 commented Nov 16, 2017

I republished the current version of gulp-webserver with #125 applied on global npm as @dsoko2/gulp-webserver for anyone, who also needs to have it fixed immediately. I will keep my fork close to the original project (if there should be updates) and deprecate it, when this issue is fixed in the original gulp-webserver on npm

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

No branches or pull requests

4 participants