simbo / auto-plug Goto Github PK

Get this when launching with a slush (gulp) file that relies on this library.

Here is the full output:
/web/node_test/slush-blueprints/node_modules/auto-plug/node_modules/defined-options/node_modules/validate-by-shorthand/lib/validate-value.js:16
if (is.string(validate)) {
^
TypeError: Property 'string' of object # is not a function
at validateValue (/web/node_test/slush-blueprints/node_modules/auto-plug/node_modules/defined-options/node_modules/validate-by-shorthand/lib/validate-value.js:16:12)
at AutoPlug.setOptions (/web/node_test/slush-blueprints/node_modules/auto-plug/lib/auto-plug.js:99:15)
at new AutoPlug (/web/node_test/slush-blueprints/node_modules/auto-plug/lib/auto-plug.js:89:10)
at module.exports (/web/node_test/slush-blueprints/node_modules/auto-plug/index.js:11:20)
at Object. (/web/node_test/slush-blueprints/tasks/categories.js:3:39)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Module.require (module.js:364:17)

Looks like one of the tests for a dependency of this library has failed (defined-options).

Just a small thing,
On the README for tip before pattern option you have a typo

require('auo-plug')

I think gulp users should be required to be explicit, i.e. require('auto-plug')('gulp').

I get that defaults are convenient, but I think in this case it leans too far towards magic. Also, it's favoritism that feels counter to the purpose of auto-plug, ie. to be 'the' way to auto-require plugins for anything. I think it should stick to its guns and be completely agnostic.