fix: exit with code 1 on crash if --exitcrash (#946)

bdentino · remy · commit facc8cb2c708 · 2017-09-04T20:16:40.000+01:00
I'm using nodemon in a docker container with a restart policy configured to restart the container in case it crashes. If the app crashes, nodemon exits with code 0 and docker assumes it was a graceful exit, so it does not restart my container. I don't see the harm in changing the exit code here to reflect an unexpected termination, given that this path is only exercised in the event of a crash.
diff --git a/lib/monitor/run.js b/lib/monitor/run.js
@@ -169,7 +169,7 @@ function run(options) {
       if (options.exitcrash) {
         utils.log.fail('app crashed');
         if (!config.required) {
-          process.exit(0);
+          process.exit(1);
         }
       } else {
         utils.log.fail('app crashed - waiting for file changes before' +

Original file line number	Diff line number	Diff line change
`@@ -169,7 +169,7 @@ function run(options) {`
`169`	`169`	`if (options.exitcrash) {`
`170`	`170`	`utils.log.fail('app crashed');`
`171`	`171`	`if (!config.required) {`
`172`		`- process.exit(0);`
	`172`	`+ process.exit(1);`
`173`	`173`	`}`
`174`	`174`	`} else {`
`175`	`175`	`utils.log.fail('app crashed - waiting for file changes before' +`