This repository has been archived by the owner on Aug 11, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 33
Catch exit by Ctrl+C #21
Comments
Indeed! This module should probably use signal-exit to remove locks. |
Good catch, I hadn't seen that module before. |
jskorepa
added a commit
to jskorepa/lockfile
that referenced
this issue
Mar 14, 2018
iarna
added a commit
to npm/npm
that referenced
this issue
May 3, 2018
Switches to `signal-exit` to detect abnormal exits and remove locks. PR-URL: npm/lockfile#29 Fixes: npm/lockfile#21 Credit: @Redsandro Reviewed-By: @isaacs
iarna
added a commit
to npm/npm
that referenced
this issue
May 4, 2018
Switches to `signal-exit` to detect abnormal exits and remove locks. PR-URL: npm/lockfile#29 Fixes: npm/lockfile#21 Credit: @Redsandro Reviewed-By: @isaacs
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
If the script is exited by Ctrl+C, the lock is not removed.
If the script exits normally, it is caught by
process.on('exit')
It should also listen to
process.on('SIGINT')
(linux at least).The text was updated successfully, but these errors were encountered: