-
Notifications
You must be signed in to change notification settings - Fork 88
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
Node process is hanging around after debugging session #191
Comments
We can possibly look at post launch tasks in vscode, or something through fake. The trick will be to find the event to hook into. |
Yup, the question is how we can stop |
@alfonsogarciacaro ideas? |
Fable 2.1 will work directly as a child of the Webpack process, not sure if that will fix but I can try. |
Hmm, I gave a quick try but in my case I couldn't even stop the debugging. I guess the issue is FAKE is running Fable no the launch.json. Not sure how it can be fixed, hopefully the VS Code replies to @isaacabraham issue. |
@alfonsogarciacaro hmmm, you should be able to explicitly stop the fake process by going to the terminal window and hitting ctrl + c. |
Yes, sorry, I can do that. What I can't do is stop the debugging by clicking the red button in the VS Code debugger control :) |
any news here? |
Taken from the linked issue:
So no :-( I've raised an issue in VS Code as well to track this. @alfonsogarciacaro what do you mean by
What should we do instead? @theimowski Am I right that the biggest issue is that people won't realise that node is running if they kill code suddenly? |
well in my case, when I stopped the debugging session, node process was still running meaning you could run neither |
Yeah, same thing. Note in VSCode that if you stop debugging and then hit F5 again, it'll simply re-attach to the same client app (it should start the server again though). |
@theimowski I'm not sure that we'll be able to get this in for v1 given that it's looking like it'll be dependent on a feature being added to Code. |
I'll try to look closer at this issue in spare time and see if there's another possible workaround |
Looking at that now and actually I didn't know one can ctrl+c the task in tasks window to stop node process. Maybe we can treat this as "by design" for now rather than "bug" until there's an option in vs code to get a handle of the "preLaunchTask" somehow. To emphasise the behaviour I think we should add an explicit note at https://safe-stack.github.io/docs/feature-debugging/ pointing to the troubleshooting section - this can be easily missed as in my case |
Marking as "Blocked" until we have a solution from VS Code |
After stopping the "Debug SAFE App" configuration, node process is still working in background.
We need to find a way to force stop it.
@isaacabraham already mentioned that issue in the PR
The text was updated successfully, but these errors were encountered: