Merge pull request #11 from coderhaoxin/ignore-node_modules

ignore node_modules

.travis.yml

@@ -1,5 +1,5 @@
 language: node_js
 node_js:
-  - "0.11"
   - "0.10"
-  - "0.8"
+  - "0.12"
+  - "iojs"

README.md

@@ -55,6 +55,7 @@ var dir = requireDir('./path/to/dir', {recurse: true});
 ## Options
 
 `recurse`: Whether to recursively `require()` subdirectories too.
+(`node_modules` within subdirectories will be ignored.)
 Default is false.
 
 `duplicates`: By default, if multiple files share the same basename, only the

index.js

@@ -69,6 +69,9 @@ module.exports = function requireDir(dir, opts) {
 
             if (FS.statSync(path).isDirectory()) {
                 if (opts.recurse) {
+                    if (base === 'node_modules') {
+                        continue;
+                    }
                     map[base] = requireDir(path, opts);
 
                     // if duplicates are wanted, key off the full name too:

package.json

@@ -6,6 +6,7 @@
 , "dependencies": {}
 , "devDependencies":
   { "coffee-script": "~1.3.3"
+  , "mkdirp": "^0.5.0"
   }
 , "engines":
   { "node": "*"

test/recurse.js

@@ -1,4 +1,8 @@
 var assert = require('assert');
+var mkdirp = require('mkdirp');
+var path = require('path');
+var fs = require('fs');
+
 var requireDir = require('..');
 
 // first test without recursing:
@@ -7,6 +11,14 @@ assert.deepEqual(requireDir('./recurse'), {
 });
 
 // then test with recursing:
+var dir = path.join(__dirname, 'recurse', 'node_modules');
+try {
+    fs.statSync(path.join(dir, 'fake.js'));
+} catch (e) {
+    mkdirp.sync(dir);
+    fs.writeFileSync(path.join(dir, 'fake.js'), 'module.exports = "ignore";');
+}
+
 assert.deepEqual(requireDir('./recurse', {recurse: true}), {
     a: 'a',
     b: {