From 3ecb57c89378a663d7fcfb95b0a105f891ae5238 Mon Sep 17 00:00:00 2001 From: Erik Arvidsson Date: Fri, 10 Aug 2018 15:34:47 -0700 Subject: [PATCH] Make sure we add the transformer too --- src/transforms/babel.js | 8 ++++++++ test/javascript.js | 4 +++- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/src/transforms/babel.js b/src/transforms/babel.js index 841d6a65ba9..4b14dfe6f27 100644 --- a/src/transforms/babel.js +++ b/src/transforms/babel.js @@ -265,6 +265,14 @@ async function getEnvPlugins(targets, useBuiltIns = false) { {}, {targets, modules: false, useBuiltIns: useBuiltIns ? 'entry' : false} ).plugins; + + // babel-preset-env version 6.x does not cover object-rest-spread so always + // add it. + plugins.push([ + require('babel-plugin-transform-object-rest-spread'), + {useBuiltIns} + ]); + envCache.set(key, plugins); return plugins; } diff --git a/test/javascript.js b/test/javascript.js index 0a9840624e9..e2a28891da5 100644 --- a/test/javascript.js +++ b/test/javascript.js @@ -29,7 +29,9 @@ describe('javascript', function() { }); it('should produce a basic JS bundle with object rest spread support', async function() { - let b = await bundle(__dirname + '/integration/object-rest-spread/object-rest-spread.js'); + let b = await bundle( + __dirname + '/integration/object-rest-spread/object-rest-spread.js' + ); assert.equal(b.assets.size, 1);