change message - options.format cannot be missing

rollup · Aug 11, 2017 · f64babd · f64babd
2 parents 2395452 + 7df44de
commit f64babd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/Bundle.js b/src/Bundle.js
@@ -502,10 +502,10 @@ export default class Bundle {
 
 			const finalise = finalisers[ options.format ];
 			if ( !finalise ) {
-				error( {
+				error({
 					code: 'INVALID_OPTION',
-					message: `You must specify an output type - valid options are ${keys( finalisers ).join( ', ' )}`
-				} );
+					message: `Invalid format: ${options.format} - valid options are ${keys( finalisers ).join( ', ' )}`
+				});
 			}
 
 			timeStart( 'render format' );

diff --git a/src/rollup/index.js b/src/rollup/index.js
@@ -98,13 +98,11 @@ export default function rollup ( options ) {
 				}
 
 				if ( !options.format ) {
-					error({ // TODO make this an error
+					error({
 						code: 'MISSING_FORMAT',
 						message: `You must supply an output format`,
 						url: `https://github.com/rollup/rollup/wiki/JavaScript-API#format`
 					});
-
-					options.format = 'es';
 				}
 
 				checkAmd( options );