rollup-plugin-babel - npm Package Compare versions

Comparing version 2.3.5 to 2.3.6

CHANGELOG.md

 # rollup-plugin-babel changelog
 
+## 2.3.6
+
+* Fix cache misses in preflight check ([#29](https://github.com/rollup/rollup-plugin-babel/pull/29))
+
 ## 2.3.5
@@ -4,0 +8,0 @@

dist/rollup-plugin-babel.cjs.js

@@ -16,5 +16,5 @@ 'use strict';
 function preflightCheck(options, dir) {
-	var helpers = undefined;
+	if (!preflightCheckResults[dir]) {
+		var helpers = undefined;
 
-	if (!preflightCheckResults[dir]) {
 		options = assign({}, options);
@@ -33,5 +33,7 @@ options.filename = path.join(dir, 'x.js');
 		}
+
+		preflightCheckResults[dir] = helpers;
 	}
 
-	return preflightCheckResults[dir] = helpers;
+	return preflightCheckResults[dir];
 }
@@ -89,3 +91,3 @@
 						if (inlineHelpers[helper]) {
-							warnOnce('The \'' + helper + ' Babel helper is used more than once in your code. It\'s strongly recommended that you use the "external-helpers-2" plugin or the "es2015-rollup" preset. See https://github.com/rollup/rollup-plugin-babel#configuring-babel for more information');
+							warnOnce('The \'' + helper + '\' Babel helper is used more than once in your code. It\'s strongly recommended that you use the "external-helpers-2" plugin or the "es2015-rollup" preset. See https://github.com/rollup/rollup-plugin-babel#configuring-babel for more information');
 						}
@@ -107,3 +109,3 @@
 
-			return babelCore.buildExternalHelpers(helpers, 'var').replace(/var babelHelpers_+\n/, '').replace(/babelHelpers\.(.+) = function(?: \w+)?/g, 'function babelHelpers_$1').replace(/babelHelpers\.(.+) = /g, 'var babelHelpers_$1 = ').replace('babelHelpers;', '') // not sure where this comes from...
+			return babelCore.buildExternalHelpers(helpers, 'var').replace(/var babelHelpers = {};\n/, '').replace(/babelHelpers\.(.+) = function(?: \w+)?/g, 'function babelHelpers_$1').replace(/babelHelpers\.(.+) = /g, 'var babelHelpers_$1 = ').replace('babelHelpers;', '') // not sure where this comes from...
 			.trim() + '\n';
@@ -110,0 +112,0 @@ }

dist/rollup-plugin-babel.es6.js

@@ -13,5 +13,5 @@ import { join, dirname } from 'path';
 function preflightCheck(options, dir) {
-	var helpers = undefined;
+	if (!preflightCheckResults[dir]) {
+		var helpers = undefined;
 
-	if (!preflightCheckResults[dir]) {
 		options = assign({}, options);
@@ -30,5 +30,7 @@ options.filename = join(dir, 'x.js');
 		}
+
+		preflightCheckResults[dir] = helpers;
 	}
 
-	return preflightCheckResults[dir] = helpers;
+	return preflightCheckResults[dir];
 }
@@ -86,3 +88,3 @@
 						if (inlineHelpers[helper]) {
-							warnOnce('The \'' + helper + ' Babel helper is used more than once in your code. It\'s strongly recommended that you use the "external-helpers-2" plugin or the "es2015-rollup" preset. See https://github.com/rollup/rollup-plugin-babel#configuring-babel for more information');
+							warnOnce('The \'' + helper + '\' Babel helper is used more than once in your code. It\'s strongly recommended that you use the "external-helpers-2" plugin or the "es2015-rollup" preset. See https://github.com/rollup/rollup-plugin-babel#configuring-babel for more information');
 						}
@@ -104,3 +106,3 @@
 
-			return buildExternalHelpers(helpers, 'var').replace(/var babelHelpers_+\n/, '').replace(/babelHelpers\.(.+) = function(?: \w+)?/g, 'function babelHelpers_$1').replace(/babelHelpers\.(.+) = /g, 'var babelHelpers_$1 = ').replace('babelHelpers;', '') // not sure where this comes from...
+			return buildExternalHelpers(helpers, 'var').replace(/var babelHelpers = {};\n/, '').replace(/babelHelpers\.(.+) = function(?: \w+)?/g, 'function babelHelpers_$1').replace(/babelHelpers\.(.+) = /g, 'var babelHelpers_$1 = ').replace('babelHelpers;', '') // not sure where this comes from...
 			.trim() + '\n';
@@ -107,0 +109,0 @@ }

package.json

 {
   "name": "rollup-plugin-babel",
-  "version": "2.3.5",
+  "version": "2.3.6",
   "description": "Seamless integration between Rollup and Babel.",
@@ -38,4 +38,4 @@ "main": "dist/rollup-plugin-babel.cjs.js",
   "devDependencies": {
+    "babel-plugin-transform-decorators-legacy": "^1.3.4",
     "babel-plugin-transform-runtime": "^6.3.13",
-    "babel-plugin-transform-decorators-legacy": "^1.3.4",
     "babel-preset-es2015": "^6.1.18",
@@ -46,3 +46,3 @@ "babel-preset-es2015-rollup": "^1.0.0",
     "mocha": "^2.3.3",
-    "rollup": "^0.22.0",
+    "rollup": "^0.24.0",
     "rollup-plugin-babel": "^2.2.0",
@@ -49,0 +49,0 @@ "source-map": "^0.5.3"

src/index.js

@@ -13,5 +13,5 @@ import { dirname, join } from 'path';
 function preflightCheck ( options, dir ) {
-	let helpers;
+	if ( !preflightCheckResults[ dir ] ) {
+		let helpers;
 
-	if ( !preflightCheckResults[ dir ] ) {
 		options = assign( {}, options );
@@ -34,5 +34,7 @@ options.filename = join( dir, 'x.js' );
 		}
+
+		preflightCheckResults[ dir ] = helpers;
 	}
 
-	return preflightCheckResults[ dir ] = helpers;
+	return preflightCheckResults[ dir ];
 }
@@ -90,3 +92,3 @@
 						if ( inlineHelpers[ helper ] ) {
-							warnOnce( `The '${helper} Babel helper is used more than once in your code. It's strongly recommended that you use the "external-helpers-2" plugin or the "es2015-rollup" preset. See https://github.com/rollup/rollup-plugin-babel#configuring-babel for more information` );
+							warnOnce( `The '${helper}' Babel helper is used more than once in your code. It's strongly recommended that you use the "external-helpers-2" plugin or the "es2015-rollup" preset. See https://github.com/rollup/rollup-plugin-babel#configuring-babel for more information` );
 						}
@@ -109,3 +111,3 @@
 			return buildExternalHelpers( helpers, 'var' )
-				.replace( /var babelHelpers.+\n/, '' )
+				.replace( /var babelHelpers = {};\n/, '' )
 				.replace( /babelHelpers\.(.+) = function(?: \w+)?/g, 'function babelHelpers_$1' )
@@ -112,0 +114,0 @@ .replace( /babelHelpers\.(.+) = /g, 'var babelHelpers_$1 = ' )

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

19173

increased by1.2%

Lines of code

266

increased by1.14%

No dependency changes