-
Notifications
You must be signed in to change notification settings - Fork 921
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
"snowpack dev" requires two Ctrl-C to exit. #368
Comments
So, I don't know if this is a Windows specific issue (I'm using Powershell on Windows 10), but I did find a sort of workaround here: Effectively creating a custom handler for the user input for Ctrl+C on Windows and then having that method process the "SIGINT" event so it gets picked up properly by the handler.
It works, but causes no confirmation message to appear - need to look into that. |
FWIW I’m on latest MacOS when reporting. |
I suppose it's possible to remove the windows check and manually handle the user input regardless of OS... I haven't tried on my Mac yet though. |
I noticed this on my windows machine as well using Git Bash |
Confirms... the "fix" also works for Mac by removing the if statement:
I read somewhere that it's possible one of the other watchers is swallowing the initial ctrl+c command from the user, but I've not looked into that yet. |
I noticed the same thing as well. Implementing the above does work on Git Bash as well. |
Sounds like that's the fix then for all environments. Would love a PR if you can add one, @mojiwa! |
Absolutely. I'll raise one this evening. |
Resolved via #380 |
Original Discussion: https://www.pika.dev/npm/snowpack/discuss/244
/cc @quasor, @acoyfellow, @FredKSchott
It should only require one.
The text was updated successfully, but these errors were encountered: