Merge pull request #191 from nlfurniss/remove-browserPolyfill-option

Remove browserPolyfill option

README.md

@@ -90,30 +90,6 @@ let scriptTree = esTranspiler(inputTree, {
 });
 ```
 
-## Polyfill
-
-In order to use some of the ES6 features you must include the Babel
-[polyfill](http://babeljs.io/docs/usage/polyfill/#usage-in-browser).
-
-You don't always need this, review which features need the polyfill here: [ES6
-Features](https://babeljs.io/docs/learn-es6).
-
-```js
-const esTranspiler = require('broccoli-babel-transpiler');
-let scriptTree = esTranspiler(inputTree, { browserPolyfill: true });
-```
-
-By default, the polyfill will appear as `./browser-polyfill.js` in the output
-tree. You can specify a custom output path and filename by supplying a (relative)
-filename in the `browserPolyfillPath` option:
-
-```js
-let scriptTree = esTranspiler(inputTree, {
-  browserPolyfill: true,
-  browserPolyfillPath: 'vendor/polyfill.js'
-})
-```
-
 ## Plugins
 
 Use of custom plugins works similarly to `babel` itself. You would pass a

index.js

@@ -41,20 +41,6 @@ function Babel(inputTree, _options) {
   options.persist = 'persist' in options ? options.persist : true;
   options.async = true;
 
-  if (options.browserPolyfill) {
-    let polyfillPath = require.resolve('@babel/polyfill/package.json');
-    polyfillPath = path.join(path.dirname(polyfillPath), "dist");
-
-    let browserPolyfillPath = options.browserPolyfillPath || "browser-polyfill.js"
-    let polyfill = funnel(polyfillPath, {
-      files: ['polyfill.js'],
-      getDestinationPath: () => browserPolyfillPath
-    });
-    inputTree = mergeTrees([polyfill, inputTree]);
-  }
-  delete options.browserPolyfill;
-  delete options.browserPolyfillPath;
-
   Filter.call(this, inputTree, options);
 
   delete options.persist;

tests/test.js

@@ -629,28 +629,6 @@ describe('filters files to transform', function() {
   });
 });
 
-describe('consume broccoli-babel-transpiler options', function() {
-  it('enabled', function() {
-    let options = {
-      browserPolyfill: true
-    };
-
-    babel = new Babel('foo', options);
-    let code = babel.processString('path', 'relativePath');
-    expect(code).to.be.ok;
-  });
-
-  it('explicitly disabled', function() {
-    let options = {
-      browserPolyfill: false
-    };
-
-    babel = new Babel('foo', options);
-    let code = babel.processString('path', 'relativePath');
-    expect(code).to.be.ok;
-  });
-});
-
 describe('when options change', function() {
   let originalHash, options, fakeConsole, consoleMessages;