Remove bundlerOptions for now

parcel-bundler · Feb 4, 2018 · 127d339 · 127d339
1 parent 43ea619
commit 127d339
Show file tree

Hide file tree

Showing 9 changed files with 10 additions and 21 deletions.
diff --git a/src/transforms/babel.js b/src/transforms/babel.js
@@ -116,22 +116,9 @@ async function getBabelConfig(asset) {
 /**
  * Finds a .babelrc for an asset. By default, .babelrc files inside node_modules are not used.
  * However, there are some exceptions:
- *   - if `bundlerOptions.babel` is set in package.json
  *   - if `browserify.transforms` includes "babelify" in package.json (for legacy module compat)
  */
 async function getBabelRc(asset) {
-  // Check for a bundlerOptions key in package.json
-  let bundlerOptions = asset.package && asset.package.bundlerOptions;
-  if (bundlerOptions && bundlerOptions.babel) {
-    // Override the config with a bundler specific one if specified as an object
-    if (typeof bundlerOptions.babel === 'object') {
-      return bundlerOptions.babel;
-    }
-
-    // Otherwise, return the .babelrc
-    return await findBabelRc(asset);
-  }
-
   // Support legacy browserify packages
   let browserify = asset.package && asset.package.browserify;
   if (browserify && Array.isArray(browserify.transform)) {

diff --git a/...ode-modules-bundleroptions/.eslintrc.json → ...el-node-modules-browserify/.eslintrc.json b/...ode-modules-bundleroptions/.eslintrc.json → ...el-node-modules-browserify/.eslintrc.json
diff --git a/...abel-node-modules-bundleroptions/index.js → ...on/babel-node-modules-browserify/index.js b/...abel-node-modules-bundleroptions/index.js → ...on/babel-node-modules-browserify/index.js
diff --git a/...-bundleroptions/node_modules/foo/.babelrc → ...ules-browserify/node_modules/foo/.babelrc b/...-bundleroptions/node_modules/foo/.babelrc → ...ules-browserify/node_modules/foo/.babelrc
diff --git a/...-bundleroptions/node_modules/foo/index.js → ...ules-browserify/node_modules/foo/index.js b/...-bundleroptions/node_modules/foo/index.js → ...ules-browserify/node_modules/foo/index.js
diff --git a/test/integration/babel-node-modules-browserify/node_modules/foo/package.json b/test/integration/babel-node-modules-browserify/node_modules/foo/package.json
diff --git a/...-node-modules-bundleroptions/package.json → ...abel-node-modules-browserify/package.json b/...-node-modules-bundleroptions/package.json → ...abel-node-modules-browserify/package.json
diff --git a/test/integration/babel-node-modules-bundleroptions/node_modules/foo/package.json b/test/integration/babel-node-modules-bundleroptions/node_modules/foo/package.json
diff --git a/test/javascript.js b/test/javascript.js
@@ -534,9 +534,9 @@ describe('javascript', function() {
     assert(!file.includes('class Bar {}'));
   });
 
-  it('should compile node_modules if bundlerOptions is set', async function() {
+  it('should compile node_modules if legacy browserify options are found', async function() {
     await bundle(
-      __dirname + '/integration/babel-node-modules-bundleroptions/index.js'
+      __dirname + '/integration/babel-node-modules-browserify/index.js'
     );
 
     let file = fs.readFileSync(__dirname + '/dist/index.js', 'utf8');