Merge pull request #24 from backflip/feature/extend

Add possibility to ignore matches via resolvePath option

Author: backflip

README.md

@@ -77,6 +77,18 @@ Parameters:
 
 The `path` package is available in this context.
 
+Returning a falsy value will ignore the resolved path:
+```javascript
+function(match, targetFile) {
+    // Ignore `/lib.js`
+    if (match.match(/\/lib\.js$/)) {
+        return null;
+    }
+
+    return path.join(path.dirname(targetFile.path), match);
+}
+```
+
 #### options.log
 Type: `Boolean`
 

index.js

@@ -48,6 +48,14 @@ function resolveDependencies(config) {
 			while (match = pattern.exec(content)) {
 				filePath = config.resolvePath(match[1], targetFile);
 
+				if (!filePath) {
+					if (config.log) {
+						Log('[' + AnsiColors.green(PLUGIN_NAME) + '] Ignored', match[1]);
+					}
+
+					continue;
+				}
+
 				// Check for circular dependencies
 				try {
 					dag.addEdge(targetFile.path, filePath);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "gulp-resolve-dependencies",
-  "version": "3.0.1",
+  "version": "4.0.0",
   "description": "Resolve dependency directives in assets (e.g. \"@requires\" or \"//= require\" in JavaScript)",
   "license": "MIT",
   "repository": "backflip/gulp-resolve-dependencies",
@@ -9,7 +9,6 @@
     "node": ">=6"
   },
   "scripts": {
-    "pretest": "npm i",
     "test": "./node_modules/mocha/bin/mocha"
   },
   "main": "./index.js",

test/expected/filtered.js

@@ -0,0 +1,15 @@
+/**
+ * @requires lib2.js/lib2.js
+ */
+console.log('lib.js');
+
+/**
+ * @requires ../libs/lib.js
+ * @requires ../libs/lib2.js/lib2.js
+ */
+console.log('test.js');
+
+/**
+ * @requires test/test.js
+ */
+console.log('main.js');

test/main.js

@@ -89,4 +89,25 @@ describe('gulp-resolve-dependencies', function() {
 				done();
 			});
 	});
+
+	it('should ignore lib2.js', function(done) {
+		gulp.src(__dirname + '/fixtures/main.js')
+			.pipe(resolveDependencies({
+				resolvePath: function(match, targetFile) {
+					// Ignore specific matches
+					if (match.match(/\/lib2\.js$/)) {
+						return null;
+					}
+					
+					return path.join(path.dirname(path.resolve(targetFile.path)), match);
+				},
+				log: true
+			}))
+			.pipe(concat('filtered.js'))
+			.pipe(gulp.dest(__dirname + '/results/'))
+			.pipe(es.wait(function() {
+				assertFilesEqual('filtered.js');
+				done();
+			}));
+	});
 });