Webpack-dev-server: Warning prevents reload

Created on 9 Feb 2017 · 5Comments · Source: webpack/webpack-dev-server

Do you want to request a feature or report a bug?

Bug

What is the current behavior?

  ⚠  http://eslint.org/docs/rules/no-console  Unexpected console statement  
  .../src/index.js:60:1
  console.log('foo');
   ^

✘ 1 problem (0 errors, 1 warning)

[WDS] App updated. Recompiling...
client?843a:40 [WDS] Errors while compiling. Reload prevented.

I've seen previous issues where this is considered a bug (such as #707) and was fixed and closed at the time.

If the current behavior is a bug, please provide the steps to reproduce.

Add a warning scenario with ESLint and trigger it.

What is the expected behavior?

The browser reloads after file changes that do not cause any actual errors.

If this is a feature request, what is motivation or use case for changing the behavior?

Please mention your webpack and Operating System version.

OS - Arch Linux
webpack - 2.2.1
webpack-dev-server - 2.3.0

Source

tquetano-r7

Most helpful comment

@mnpenner https://github.com/MoOx/eslint-loader#emitwarning-default-false

{
  enforce: 'pre',
  exclude: /node_modules/,
  test: /\.jsx?$/,
  use: {
    loader: 'eslint-loader',
    options: {
      emitWarning: true, // @note: only for development
    },
  },
},

chemoish on 19 Oct 2017

👍3

All 5 comments

I don't think this is a bug in webpack-dev-server. Apparently eslint-loader passes this eslint warning as an error (I think, haven't actually checked). As you can see here, it only shows that "Reload prevented" message when there is an actual compile error.

SpaceK33z on 9 Feb 2017

Nope I am just an idiot ... i had emitError set to true in the configuration, which was always emitting errors! Derp.

Closing.

tquetano-r7 on 9 Feb 2017

@tquetano-r7 Where specifically did you have emitError? In .eslintrc.js or somewhere else?

I can't find that setting in either my webpack.config.js nor .eslintrc.js but this is the behaviour I'm observing.

mnpenner on 7 Sep 2017

@mnpenner https://github.com/MoOx/eslint-loader#emitwarning-default-false

{
  enforce: 'pre',
  exclude: /node_modules/,
  test: /\.jsx?$/,
  use: {
    loader: 'eslint-loader',
    options: {
      emitWarning: true, // @note: only for development
    },
  },
},

chemoish on 19 Oct 2017

👍3

Errors while compiling. Reload prevented.