Remove reliance on the _exit pseudo-config in error handler

isaacs · isaacs · commit 9f200abb94ea · 2020-08-24T14:38:38.000-07:00
This flag no longer exists.  We just always exit now when we're done.
diff --git a/lib/utils/error-handler.js b/lib/utils/error-handler.js
@@ -77,23 +77,17 @@ process.on('exit', code => {
     wroteLogFile = false
   }
 
-  const doExit = npm.config && npm.config.loaded && npm.config.get('_exit')
-  if (doExit) {
-    // actually exit.
-    if (exitCode === 0 && !itWorked) {
-      exitCode = 1
-    }
-    if (exitCode !== 0) process.exit(exitCode)
-  } else {
-    itWorked = false // ready for next exit
+  // actually exit.
+  if (exitCode === 0 && !itWorked) {
+    exitCode = 1
   }
+  if (exitCode !== 0) process.exit(exitCode)
 })
 
 const exit = (code, noLog) => {
   exitCode = exitCode || process.exitCode || code
 
-  const doExit = npm.config && npm.config.loaded ? npm.config.get('_exit') : true
-  log.verbose('exit', [code, doExit])
+  log.verbose('exit', code)
   if (log.level === 'silent') noLog = true
 
   const reallyExit = (er) => {
