Merge pull request #140 from geotrev/js/terser

Update rollup to use terser instead of uglify for minification

Author: geotrev

package-lock.json

@@ -96,12 +96,12 @@
     "rollup": "^1.16.3",
     "rollup-plugin-babel": "^4.3.2",
     "rollup-plugin-node-resolve": "^5.2.0",
+    "rollup-plugin-terser": "^5.1.1",
     "rollup-plugin-uglify": "^6.0.2",
     "sass-loader": "^7.1.0",
     "style-loader": "^0.23.1",
     "terser-webpack-plugin": "^1.3.0",
     "uglify-js": "^3.6.0",
-    "uglifyjs-webpack-plugin": "^2.1.2",
     "webpack": "^4.35.0",
     "webpack-cli": "^3.3.0",
     "webpack-dev-server": "^3.7.2",

package.json

@@ -1,4 +1,4 @@
-import { uglify } from "rollup-plugin-uglify"
+import { terser } from "rollup-plugin-terser"
 import path from "path"
 import resolve from "rollup-plugin-node-resolve"
 import babel from "rollup-plugin-babel"
@@ -55,13 +55,14 @@ const umdMinOutput = Object.assign({}, umdOutput, {
 const umdMinPlugins = []
 umdMinPlugins.push(
   ...plugins,
-  uglify({
+  terser({
     output: {
       comments: (_, comment) => {
-        if (comment.type === "comment2") {
-          return /@preserve|@license|@cc_on/i.test(comment.value)
+        const { value, type } = comment
+
+        if (type === "comment2") {
+          return /@preserve|@license|@cc_on/i.test(value)
         }
-        return false
       },
     },
     mangle: { reserved: ["Undernet"] },
@@ -74,4 +75,4 @@ const umdMinBundle = {
   plugins: umdMinPlugins,
 }
 
-module.exports = [umdBundle, umdMinBundle]
+export default [umdBundle, umdMinBundle]

rollup.config.js

@@ -9,10 +9,10 @@
 @import "config";
 @import "utilities/mixins";
 
-// remove this reset if you are scoping!
+// .#{$scope} {
+// Reset within the scope is optional
 @import "utilities/reset";
 
-// .#{$scope} {
 @import "utilities/classes";
 
 @import "layout/grid";