don't return error when nothing to commit. (#148)

ttwthomas · stephenlacy · commit af32e231b16d · 2017-02-08T14:16:24.000-07:00
* don't return error when nothing to commit. same as #118 * Reduces length and removes the need for two checks
diff --git a/lib/commit.js b/lib/commit.js
@@ -83,7 +83,7 @@ module.exports = function(message, opt) {
     }
 
     var execChildProcess = exec(cmd, opt, function(err, stdout, stderr) {
-      if (err) return cb(err);
+      if (err && (String(stdout).indexOf('no changes added to commit') === 0)) return cb(err);
       if (!opt.quiet) gutil.log(stdout, stderr);
       files.forEach(self.push.bind(self));
       self.emit('end');

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ module.exports = function(message, opt) {`
`83`	`83`	`}`
`84`	`84`
`85`	`85`	`var execChildProcess = exec(cmd, opt, function(err, stdout, stderr) {`
`86`		`- if (err) return cb(err);`
	`86`	`+ if (err && (String(stdout).indexOf('no changes added to commit') === 0)) return cb(err);`
`87`	`87`	`if (!opt.quiet) gutil.log(stdout, stderr);`
`88`	`88`	`files.forEach(self.push.bind(self));`
`89`	`89`	`self.emit('end');`