Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Merge pull request #8917 from evq/disable-notifier
Browse files Browse the repository at this point in the history
Add env var to disable webpack-notifier in build, fixes #8916
  • Loading branch information
bbondy authored May 26, 2017
2 parents c4b1e0e + e2a7998 commit aa91a43
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ var env = process.env.NODE_ENV === 'production' ? 'production'
: (process.env.NODE_ENV === 'test' ? 'test' : 'development')

function config () {
return {
let c = {
devtool: '#source-map',
cache: true,
module: {
Expand Down Expand Up @@ -55,7 +55,6 @@ function config () {
'electron': 'chrome'
},
plugins: [
new WebpackNotifierPlugin({title: 'Brave-' + env}),
new webpack.IgnorePlugin(/^\.\/stores\/appStore$/),
new webpack.IgnorePlugin(/^spellchecker/),
new webpack.DefinePlugin({
Expand All @@ -72,6 +71,10 @@ function config () {
fs: 'empty'
}
}
if (!process.env.DISABLE_WEBPACK_NOTIFIER) {
c.plugins.push(new WebpackNotifierPlugin({title: 'Brave-' + env}))
}
return c
}

function watchOptions () {
Expand Down

0 comments on commit aa91a43

Please # to comment.