Small refactoring on cleanup option

sindresorhus · May 30, 2019 · 739f683 · 739f683
1 parent e5ed8de
commit 739f683
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/index.js b/index.js
@@ -267,10 +267,8 @@ const execa = (file, args, options) => {
 		return promise;
 	})();
 
-	const processDone = new Promise(resolve => {
+	const processDone = pFinally(new Promise(resolve => {
 		spawned.on('exit', (code, signal) => {
-			cleanup();
-
 			if (timedOut) {
 				resolvable.resolve([
 					{code, signal}, '', '', ''
@@ -281,17 +279,15 @@ const execa = (file, args, options) => {
 		});
 
 		spawned.on('error', error => {
-			cleanup();
 			resolve({error});
 		});
 
 		if (spawned.stdin) {
 			spawned.stdin.on('error', error => {
-				cleanup();
 				resolve({error});
 			});
 		}
-	});
+	}), cleanup);
 
 	function destroy() {
 		if (spawned.stdout) {