Merge pull request #445 from stasm/remove-some-babel

stasm · web-flow · commit 00a1404a4c20 · 2020-01-13T14:46:08.000+01:00
Remove unneeded Babel plugins
diff --git a/README.md b/README.md
@@ -78,8 +78,7 @@ Hacking on `fluent.js` is easy! To quickly get started clone the repo:
     $ git clone https://github.com/projectfluent/fluent.js.git
     $ cd fluent.js
 
-You'll need node.js 8 LTS (v8.9.0 and up). Older 8.x versions and 7.x
-versions have been reported to work as well. node.js 6.x is not supported.
+You'll need at least Node.js 10 LTS. Older versions are not supported.
 
 Install the build tools used by all packages (Babel, Rollup, Mocha etc.):
 
diff --git a/babel_config.js b/babel_config.js
@@ -14,7 +14,7 @@ export default {
           "Safari >= 10.1",
           "iOS >= 10.3",
         ],
-        node: "8.9",
+        node: "10.0.0",
       },
       exclude: [
         // Exclude regeneratorRuntime explicitly because babel-preset-env
@@ -23,11 +23,5 @@ export default {
         "transform-regenerator"
       ],
     }]
-  ],
-  plugins: [
-    // Shipped in Firefox 57, Chrome 63
-    "@babel/plugin-proposal-async-generator-functions",
-    // Shipped in Firefox 55, Chrome 60
-    "@babel/plugin-proposal-object-rest-spread"
   ]
 };
diff --git a/fluent-bundle/makefile b/fluent-bundle/makefile
@@ -8,7 +8,7 @@ build: index.js compat.js
 test:
 	@nyc --reporter=text --reporter=html mocha \
 	    --recursive --ui tdd \
-	    --require $(ROOT)/mocha_setup \
+	    --require esm \
 	    --require ./test/index \
 	    test/**/*_test.js
 
diff --git a/fluent-dedent/makefile b/fluent-dedent/makefile
@@ -8,7 +8,7 @@ build: index.js compat.js
 test:
 	@nyc --reporter=text --reporter=html mocha \
 	    --recursive --ui tdd \
-	    --require $(ROOT)/mocha_setup \
+	    --require esm \
 	    test/**/*_test.js
 
 index.js: $(SOURCES)
diff --git a/fluent-dom/makefile b/fluent-dom/makefile
@@ -7,7 +7,7 @@ include ../common.mk
 test:
 	@nyc --reporter=text --reporter=html mocha \
 	    --recursive --ui tdd \
-	    --require $(ROOT)/mocha_setup \
+	    --require esm \
 	    --require ./test/index \
 	    test/**/*_test.js
 
diff --git a/fluent-gecko/makefile b/fluent-gecko/makefile
@@ -9,7 +9,7 @@ version = $(1)@$(shell node -e "\
 test:
 	@nyc --reporter=text --reporter=html mocha \
 	    --recursive --ui tdd \
-	    --require $(ROOT)/mocha_setup \
+	    --require esm \
 	    test/**/*_test.js
 
 build: Fluent.jsm FluentSyntax.jsm Localization.jsm DOMLocalization.jsm l10n.js fluent-react.js
diff --git a/fluent-langneg/makefile b/fluent-langneg/makefile
@@ -6,7 +6,7 @@ include ../common.mk
 test:
 	@nyc --reporter=text --reporter=html mocha \
 	    --recursive --ui tdd \
-	    --require $(ROOT)/mocha_setup \
+	    --require esm \
 	    test/**/*_test.js
 
 build: index.js compat.js
diff --git a/fluent-react/babel.config.js b/fluent-react/babel.config.js
@@ -3,14 +3,13 @@ module.exports = {
   "presets": [
     "@babel/preset-react",
     ["@babel/preset-env", {
-      "targets": "node >= 8.9.0"
+      "targets": "node >= 10.0.0"
     }]
   ],
   "plugins": [
     ["babel-plugin-transform-rename-import", {
       "original": "fluent",
       "replacement": "fluent/compat"
-    }],
-    "@babel/plugin-proposal-async-generator-functions"
+    }]
   ],
 };
diff --git a/fluent-react/compat_config.js b/fluent-react/compat_config.js
@@ -9,7 +9,6 @@ export default {
     babel({
       ...babelConfig,
       plugins: [
-        ...babelConfig.plugins,
         ["babel-plugin-transform-rename-import", {
           replacements: [
             {
diff --git a/fluent-react/package.json b/fluent-react/package.json
@@ -55,7 +55,6 @@
     "react": "^0.14.9 || ^15.0.0 || ^16.0.0"
   },
   "devDependencies": {
-    "@babel/plugin-proposal-async-generator-functions": "^7.2.0",
     "@babel/preset-env": "^7.5.5",
     "@babel/preset-react": "7.0.0",
     "babel-jest": "^24.8.0",
diff --git a/fluent-sequence/makefile b/fluent-sequence/makefile
@@ -6,7 +6,7 @@ include ../common.mk
 test:
 	@nyc --reporter=text --reporter=html mocha \
 	    --recursive --ui tdd \
-	    --require $(ROOT)/mocha_setup \
+	    --require esm \
 	    test/**/*_test.js
 
 build: index.js compat.js
diff --git a/fluent-syntax/makefile b/fluent-syntax/makefile
@@ -6,7 +6,7 @@ include ../common.mk
 test:
 	@nyc --reporter=text --reporter=html mocha \
 	    --recursive --ui tdd \
-	    --require $(ROOT)/mocha_setup \
+	    --require esm \
 	    test/**/*_test.js
 
 build: index.js compat.js
diff --git a/mocha_setup.js b/mocha_setup.js
diff --git a/package.json b/package.json
@@ -7,12 +7,7 @@
   },
   "devDependencies": {
     "@babel/core": "^7.4.3",
-    "@babel/plugin-proposal-async-generator-functions": "^7.2.0",
-    "@babel/plugin-proposal-object-rest-spread": "^7.4.3",
-    "@babel/plugin-transform-modules-commonjs": "^7.4.3",
-    "@babel/polyfill": "^7.4.3",
     "@babel/preset-env": "^7.4.3",
-    "@babel/register": "^7.4.0",
     "babel-eslint": "^10.0.1",
     "colors": "^1.3.3",
     "commander": "^2.20",

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,6 @@ export default {`
`9`	`9`	`babel({`
`10`	`10`	`...babelConfig,`
`11`	`11`	`plugins: [`
`12`		`- ...babelConfig.plugins,`
`13`	`12`	`["babel-plugin-transform-rename-import", {`
`14`	`13`	`replacements: [`
`15`	`14`	`{`