Merge pull request #75 from satoshilabs/better-webpack-errors

Added OS notification when webpack failed to compile the project

package.json

@@ -34,8 +34,8 @@
         "ethereumjs-tx": "^1.3.3",
         "ethereumjs-units": "^0.2.0",
         "ethereumjs-util": "^5.1.4",
-        "git-revision-webpack-plugin": "^3.0.3",
         "flow-webpack-plugin": "^1.2.0",
+        "git-revision-webpack-plugin": "^3.0.3",
         "hdkey": "^0.8.0",
         "html-webpack-plugin": "^3.2.0",
         "http-server": "^0.11.1",
@@ -67,6 +67,7 @@
         "trezor-connect": "^5.0.32",
         "web3": "1.0.0-beta.35",
         "webpack": "^4.16.3",
+        "webpack-build-notifier": "^0.1.29",
         "webpack-bundle-analyzer": "^2.13.1",
         "whatwg-fetch": "^2.0.4",
         "yarn-run-all": "^3.1.1"

webpack/dev.babel.js

@@ -2,8 +2,10 @@ import webpack from 'webpack';
 import GitRevisionPlugin from 'git-revision-webpack-plugin';
 import HtmlWebpackPlugin from 'html-webpack-plugin';
 import FlowWebpackPlugin from 'flow-webpack-plugin';
+import WebpackBuildNotifierPlugin from 'webpack-build-notifier';
 
-import { BundleAnalyzerPlugin } from 'webpack-bundle-analyzer';
+// turn on for bundle analyzing
+// import { BundleAnalyzerPlugin } from 'webpack-bundle-analyzer';
 
 import {
     SRC, BUILD, PORT, PUBLIC,
@@ -93,6 +95,10 @@ module.exports = {
         hints: false,
     },
     plugins: [
+        new WebpackBuildNotifierPlugin({
+            title: 'Trezor Wallet',
+            suppressSuccess: true,
+        }),
         new webpack.DefinePlugin({
             COMMITHASH: JSON.stringify(gitRevisionPlugin.commithash()),
         }),
@@ -106,11 +112,11 @@ module.exports = {
             inject: true,
             favicon: `${SRC}images/favicon.ico`,
         }),
-        new BundleAnalyzerPlugin({
-            openAnalyzer: false,
-            analyzerMode: false, // turn on to generate bundle pass 'static'
-            reportFilename: 'bundle-report.html',
-        }),
+        // new BundleAnalyzerPlugin({
+        //     openAnalyzer: false,
+        //     analyzerMode: false, // turn on to generate bundle pass 'static'
+        //     reportFilename: 'bundle-report.html',
+        // }),
         new webpack.optimize.OccurrenceOrderPlugin(),
         new webpack.NoEmitOnErrorsPlugin(),
         new webpack.HotModuleReplacementPlugin(),

webpack/production.babel.js

@@ -3,7 +3,6 @@ import webpack from 'webpack';
 import GitRevisionPlugin from 'git-revision-webpack-plugin';
 import HtmlWebpackPlugin from 'html-webpack-plugin';
 import CopyWebpackPlugin from 'copy-webpack-plugin';
-import FlowWebpackPlugin from 'flow-webpack-plugin';
 import { SRC, BUILD, PUBLIC } from './constants';
 
 const gitRevisionPlugin = new GitRevisionPlugin();
@@ -68,7 +67,6 @@ module.exports = {
             'process.env.BUILD': JSON.stringify(process.env.BUILD),
             COMMITHASH: JSON.stringify(gitRevisionPlugin.commithash()
         }),
-        new FlowWebpackPlugin(),
         new HtmlWebpackPlugin({
             chunks: ['index'],
             template: `${SRC}index.html`,

yarn.lock

@@ -7058,7 +7058,7 @@ node-libs-browser@^2.0.0:
     util "^0.10.3"
     vm-browserify "0.0.4"
 
-node-notifier@^5.2.1:
+node-notifier@5.2.1, node-notifier@^5.2.1:
   version "5.2.1"
   resolved "https://registry.yarnpkg.com/node-notifier/-/node-notifier-5.2.1.tgz#fa313dd08f5517db0e2502e5758d664ac69f9dea"
   dependencies:
@@ -10970,6 +10970,14 @@ webpack-addons@^1.1.5:
   dependencies:
     jscodeshift "^0.4.0"
 
+webpack-build-notifier@^0.1.29:
+  version "0.1.29"
+  resolved "https://registry.yarnpkg.com/webpack-build-notifier/-/webpack-build-notifier-0.1.29.tgz#d71f89bb94346c6b748e07aa3d117d2beb0a151f"
+  dependencies:
+    ansi-regex "^2.0.0"
+    node-notifier "5.2.1"
+    strip-ansi "^3.0.1"
+
 webpack-bundle-analyzer@^2.13.1:
   version "2.13.1"
   resolved "https://registry.yarnpkg.com/webpack-bundle-analyzer/-/webpack-bundle-analyzer-2.13.1.tgz#07d2176c6e86c3cdce4c23e56fae2a7b6b4ad526"