fix: canceling lint-staged via SIGINT restores state and cleans up (#881)

iiroj · web-flow · commit b078324d5e91 · 2020-06-03T17:38:15.000+03:00
When replacing listr with listr2, this was a change in default behaviour that was left unnoticed
diff --git a/lib/runAll.js b/lib/runAll.js
@@ -119,6 +119,7 @@ const runAll = async (
     ctx,
     exitOnError: false,
     nonTTYRenderer: 'verbose',
+    registerSignalListeners: false,
     ...getRenderer({ debug, quiet }),
   }
 
diff --git a/test/index2.spec.js b/test/index2.spec.js
@@ -33,6 +33,7 @@ describe('lintStaged', () => {
         },
         "exitOnError": false,
         "nonTTYRenderer": "verbose",
+        "registerSignalListeners": false,
         "renderer": "silent",
       }
     `)
@@ -58,6 +59,7 @@ describe('lintStaged', () => {
         },
         "exitOnError": false,
         "nonTTYRenderer": "verbose",
+        "registerSignalListeners": false,
         "renderer": "verbose",
       }
     `)

Original file line number	Diff line number	Diff line change
`@@ -119,6 +119,7 @@ const runAll = async (`
`119`	`119`	`ctx,`
`120`	`120`	`exitOnError: false,`
`121`	`121`	`nonTTYRenderer: 'verbose',`
	`122`	`+ registerSignalListeners: false,`
`122`	`123`	`...getRenderer({ debug, quiet }),`
`123`	`124`	`}`
`124`	`125`