Add allowNonzeroExitCode option (#176)

Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>

Author: peterchu999

index.d.ts

@@ -42,6 +42,15 @@ declare namespace open {
 		@default false
 		*/
 		readonly url?: boolean;
+
+		/**
+		Allow the opened app to exit with nonzero exit code when the `wait` option is `true`.
+		
+		We do not recommend setting this option. The convention for success is exit code zero.
+
+		@default false
+		*/
+		readonly allowNonzeroExitCode?: boolean;
 	}
 }
 

index.js

@@ -29,6 +29,7 @@ module.exports = async (target, options) => {
 		wait: false,
 		background: false,
 		url: false,
+		allowNonzeroExitCode: false,
 		...options
 	};
 
@@ -148,7 +149,7 @@ module.exports = async (target, options) => {
 			subprocess.once('error', reject);
 
 			subprocess.once('close', exitCode => {
-				if (exitCode > 0) {
+				if (options.allowNonzeroExitCode && exitCode > 0) {
 					reject(new Error(`Exited with code ${exitCode}`));
 					return;
 				}

readme.md

@@ -102,6 +102,15 @@ We do not recommend using it on targets that are not URLs.
 
 Especially useful when dealing with the [double-quotes on Windows](#double-quotes-on-windows) caveat.
 
+##### allowNonzeroExitCode
+
+Type: `boolean`\
+Default: `false`
+
+Allow the opened app to exit with nonzero exit code when the `wait` option is `true`.
+		
+We do not recommend setting this option. The convention for success is exit code zero.
+
 ## Caveats
 
 ### Double-quotes on Windows