Breaking: Remove includeContent option

gulpjs · Jun 17, 2017 · 571fbcf · 571fbcf
1 parent f6c2765
commit 571fbcf
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 15 deletions.
diff --git a/index.js b/index.js
@@ -76,9 +76,7 @@ function write(file, options, callback) {
 	}
 
 	// Set defaults for options if unset
-	var opts = defaults({}, options, {
-		includeContent: true,
-	});
+	var opts = defaults({}, options);
 
 	var sourceMap = file.sourceMap;
 

diff --git a/lib/helpers.js b/lib/helpers.js
@@ -215,10 +215,6 @@ function commentFormatter(extname, url) {
 }
 
 function includeContent(file, state, options, callback) {
-  if (!options.includeContent) {
-    delete state.sourceMap.sourcesContent;
-    return callback();
-  }
 
   state.sourceMap.sourcesContent = state.sourceMap.sourcesContent || [];
 

diff --git a/test/write.js b/test/write.js
@@ -244,14 +244,6 @@ describe('write', function() {
 		});
 	});
 
-	it('should not include source content with option includeContent=false', function(done) {
-		var file = makeFile();
-		sourcemaps.write(file, { includeContent: false }, function(err, updatedFile) {
-			expect(updatedFile.sourceMap.sourcesContent).toBe(undefined);
-			done(err);
-		});
-	});
-
 	it('should fetch missing sourceContent', function(done) {
 		var file = makeFile();
 		delete file.sourceMap.sourcesContent;