Merge pull request #108 from dotnil/fix-no-deps

fgnass · fgnass · commit 6d4b5038d783 · 2015-10-23T12:31:08.000+02:00
Fix: --no-deps is treated as nodeArgs
diff --git a/bin/node-dev b/bin/node-dev
@@ -17,7 +17,8 @@ var devArgs = process.argv.slice(2, scriptIndex)
 
 var nodeArgs = []
 var opts = minimist(devArgs, {
-  boolean: ['all-deps', 'no-deps', 'dedupe', 'poll'],
+  boolean: ['all-deps', 'deps', 'dedupe', 'poll'],
+  default: { deps: true },
   unknown: function(arg) {
     nodeArgs.push(arg)
   }
diff --git a/lib/cfg.js b/lib/cfg.js
@@ -22,7 +22,7 @@ module.exports = function(main, opts) {
   if (opts) {
     // Overwrite with CLI opts ...
     if (opts.allDeps) c.deps = -1
-    if (opts.noDeps) c.deps = 0
+    if (!opts.deps) c.deps = 0
     if (opts.dedupe) c.dedupe = true
   }
 

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,8 @@ var devArgs = process.argv.slice(2, scriptIndex)`
`17`	`17`
`18`	`18`	`var nodeArgs = []`
`19`	`19`	`var opts = minimist(devArgs, {`
`20`		`- boolean: ['all-deps', 'no-deps', 'dedupe', 'poll'],`
	`20`	`+ boolean: ['all-deps', 'deps', 'dedupe', 'poll'],`
	`21`	`+ default: { deps: true },`
`21`	`22`	`unknown: function(arg) {`
`22`	`23`	`nodeArgs.push(arg)`
`23`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ module.exports = function(main, opts) {`
`22`	`22`	`if (opts) {`
`23`	`23`	`// Overwrite with CLI opts ...`
`24`	`24`	`if (opts.allDeps) c.deps = -1`
`25`		`- if (opts.noDeps) c.deps = 0`
	`25`	`+ if (!opts.deps) c.deps = 0`
`26`	`26`	`if (opts.dedupe) c.dedupe = true`
`27`	`27`	`}`
`28`	`28`