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

[QUESTION] NPM Scripts double SIGINT with CTRL+C - is this intentional? #1591

Closed
kellyjandrews opened this issue Jul 31, 2020 · 1 comment
Closed
Labels
Awaiting Information further information is requested Release 6.x work is associated with a specific npm 6 release

Comments

@kellyjandrews
Copy link

What / Why

I've run into some issues recently where running a node process using an npm script and using CTRL+C to exit will fire the command twice. It's something I had not encountered before, but unsure if this is intentional with npm scripts, or actually considered a bug.

Where

This happened specifically with regard to the Firebase CLI. I filed an issue here - firebase/firebase-tools#2507

i  emulators: Received SIGINT (Ctrl-C) for the first time. Starting a clean shutdown.
i  emulators: Please wait for a clean shutdown or send the SIGINT (Ctrl-C) signal again to stop right now.
i  Automatically exporting data using --export-on-exit "./data" please wait for the export to finish...

⚠  emulators: Received SIGINT (Ctrl-C) 2 times. You have forced the Emulator Suite to exit without waiting for 2 subprocesses to finish. These processes may still be running on your machine:

I have tested this multiple times, and it is not a keyboard error or mistyped - just get's sent twice.

Who

  • n/a

References

  • n/a
@darcyclarke
Copy link
Contributor

npm v6 is no longer in active development; We will continue to push security releases to v6 at our team's discretion as-per our Support Policy.

If your bug is preproducible on v7, please re-file this issue using our new issue template.

If your issue was a feature request, please consider opening a new RRFC or RFC. If your issue was a question or other idea that was not CLI-specific, consider opening a discussion on our feedback repo

Closing: This is an automated message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Information further information is requested Release 6.x work is associated with a specific npm 6 release
Projects
None yet
Development

No branches or pull requests

2 participants