Disable options.ignoreCircularDependencies by default.

Author: backflip

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.0.0
+
+- Disable ```options.ignoreCircularDependencies``` by default.
+
 ## 1.1.0
 
 - Throw an error when detecting a circular dependency (thanks to @huang64). Add option ```ignoreCircularDependencies``` to disable this behavior.

README.md

@@ -58,7 +58,7 @@ Circular dependencies are either silently ignored or emit an error (thanks to [@
 #### options.pattern
 Type: `RegExp`
 
-The matching pattern (defaults to ```/\* @requires [\s-]*(.*?\.js)/g``).
+The matching pattern (defaults to ```/\* @requires [\s-]*(.*?\.js)/g```).
 
 #### options.log
 Type: `Boolean`
@@ -68,5 +68,5 @@ Whether to log the resolved dependencies (defaults to ```false```).
 #### options.ignoreCircularDependencies
 Type: `Boolean`
 
-Whether to just continue instead of emitting an error if circular dependencies are detected (defaults to ```true```).
+Whether to just continue instead of emitting an error if circular dependencies are detected (defaults to ```false```).
 

index.js

@@ -13,7 +13,7 @@ function resolveDependencies(config) {
 	var defaults = {
 			pattern: /\* @requires [\s-]*(.*\.js)/g,
 			log: false,
-			ignoreCircularDependencies: true
+			ignoreCircularDependencies: false
 		},
 		stream,
 		dag = new DAG(),

package.json

@@ -1,6 +1,6 @@
 {
   "name": "gulp-resolve-dependencies",
-  "version": "1.1.0",
+  "version": "2.0.0",
   "description": "Resolve dependency directives in assets (e.g. \"@requires\" or \"//= require\" in JavaScript)",
   "license": "MIT",
   "repository": "backflip/gulp-resolve-dependencies",

test/main.js

@@ -26,9 +26,7 @@ describe('gulp-resolve-dependencies', function() {
 
 	it('should throw error due to circular dependency', function(done) {
 		gulp.src(__dirname + '/circular/a.js')
-			.pipe(resolveDependencies({
-				ignoreCircularDependencies: false
-			}))
+			.pipe(resolveDependencies())
 			.on('error', function() {
 				done();
 			});