Merge pull request #27 from lennym/feature/filter-option

Support a filter option to exclude particular files

README.md

@@ -83,6 +83,14 @@ be the same by default, but specifying `duplicates: true` would yield:
 }
 ```
 
+`filter`: Apply a filter on the filename before require-ing. For example:
+
+```js
+requiredir('./dir', function (f) { return process.env.NODE_ENV !== 'production' && !f.match(/$dev/); })
+```
+
+This will ignore files prefixed with `dev` if running in a production environment.
+
 There might be more options in the future. ;)
 
 ## Tips

index.js

@@ -17,6 +17,8 @@ module.exports = function requireDir(dir, opts) {
     dir = dir || '.';
     opts = opts || {};
 
+    opts.filter = opts.filter || function (file) { return true; };
+
     // resolve the path to an absolute one:
     dir = Path.resolve(parentDir, dir);
 
@@ -66,6 +68,10 @@ module.exports = function requireDir(dir, opts) {
             if (path === parentFile) {
                 continue;
             }
+            // apply file filter:
+            if (!opts.filter(path)) {
+                continue;
+            }
 
             if (FS.statSync(path).isDirectory()) {
                 if (opts.recurse) {

test/filter.js

@@ -0,0 +1,9 @@
+var assert = require('assert');
+var requireDir = require('..');
+
+// filter the results to a particular file:
+assert.deepEqual(requireDir('./simple', { filter: function (filename) { return filename.match(/a\.js$/); } }), {
+    a: 'a'
+});
+
+console.log('Filter tests passed.');