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

remove killall node #108

Open
jaredatron opened this issue Aug 6, 2020 · 3 comments
Open

remove killall node #108

jaredatron opened this issue Aug 6, 2020 · 3 comments

Comments

@jaredatron
Copy link

killing every running node process seems aggressive. Why do this

"shell_cmd": "killall node; /usr/bin/env node $file"
?

@varp
Copy link
Collaborator

varp commented Aug 7, 2020

I think, historically it was done by original author of the plugin for the simplicity and after that was leaved as is. I will see, how I could enhance this. Thanks!

@tanepiper
Copy link
Owner

Correct 😄 This was way before docker and I tended to be doing a lot of multiple demonised services running - this was a shorthand way to stop them all

@sleetdrop
Copy link

Nowadays, it will kill LSP for javascript and typescript either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants