Merge pull request #32 from ngoldman/proper-exits

cli: exit 1 when appropriate

Author: ungoldman

CHANGELOG.md

@@ -3,6 +3,10 @@ All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 This change log adheres to [keepachangelog.com](http://keepachangelog.com).
 
+## Unreleased
+### Fixed
+* exit process with code 1 on aborted release or invalid directory
+
 ## [1.1.7] - 2015-04-07
 ### Fixed
 * If cli is successful, should exit with code of 0

bin/cli.js

@@ -83,7 +83,7 @@ function checkDir (workpath) {
   if (!pkgExists || !logExists) {
     console.log('Must be run in a directory with package.json and CHANGELOG.md')
     yargs.showHelp()
-    process.exit(0)
+    process.exit(1)
   }
 
   authenticate()

index.js

@@ -59,7 +59,7 @@ function ghRelease (options, auth, callback) {
     if (options.dryRun) process.exit(0)
 
     inquirer.prompt(questions, function (answers) {
-      if (!answers.confirm) return process.exit(0)
+      if (!answers.confirm) return process.exit(1)
 
       var checkCommitForm = {
         user: releaseOptions.owner,