Merge pull request #812 from rollup/disable-indent

Change indent default to false

src/utils/getIndentString.js

@@ -1,5 +1,5 @@
 export default function getIndentString ( magicString, options ) {
-	if ( !( 'indent' in options ) || options.indent === true ) {
+	if ( options.indent === true ) {
 		return magicString.getIndentString();
 	}
 

test/cli/banner-intro-outro-footer/_config.js

@@ -1,4 +1,4 @@
 module.exports = {
 	description: 'adds banner/intro/outro/footer',
-	command: 'rollup -i main.js -f iife --banner "// banner" --intro "// intro" --outro "// outro" --footer "// footer"'
+	command: 'rollup -i main.js -f iife --indent --banner "// banner" --intro "// intro" --outro "// outro" --footer "// footer"'
 };

test/cli/module-name/_config.js

@@ -1,4 +1,4 @@
 module.exports = {
 	description: 'generates UMD export with correct moduleName',
-	command: 'rollup main.js --format umd --name myBundle'
+	command: 'rollup main.js --format umd --name myBundle --indent'
 };

test/cli/no-strict/_config.js

@@ -1,4 +1,4 @@
 module.exports = {
 	description: 'use no strict option',
-	command: 'rollup -i main.js -f iife --no-strict'
+	command: 'rollup -i main.js -f iife --no-strict --indent'
 };

test/cli/no-treeshake/_config.js

@@ -1,4 +1,4 @@
 module.exports = {
 	description: 'generates IIFE export with all code',
-	command: 'rollup main.js --format iife --name shakeless --no-treeshake'
+	command: 'rollup main.js --format iife --name shakeless --no-treeshake --indent'
 };

test/test.js

@@ -274,6 +274,12 @@ describe( 'rollup', function () {
 			const config = loadConfig( FORM + '/' + dir + '/_config.js' );
 
 			if ( config.skipIfWindows && process.platform === 'win32' ) return;
+			if ( !config.options ) {
+				config.options = {};
+			}
+			if ( !( 'indent' in config.options ) ) {
+				config.options.indent = true;
+			}
 
 			const options = extend( {}, {
 				entry: FORM + '/' + dir + '/main.js',