Use webpack-dev-middleware? about react-hot-boilerplate HOT 3 CLOSED

My 👎 on this is that it adds more code to the boilerplate that isn't strictly needed which you might say contradicts the following snippet from the readme:

This boilerplate is purposefully simple to show the minimal configuration for React Hot Loader.

from react-hot-boilerplate.

@danmartinez101 but don't you lose out on the "syntax error overlay" that @gaearon mentioned? It's very useful and having at least an example of using the middleware would be beneficial IMO.

from react-hot-boilerplate.

The overlay is a very nice feature indeed! I skimmed over that part. Sorry for that :/
Thank you for calling me out on this :)

While my comment was wrong, the spirit of it still holds true: I think it would be best to remove as much code from this boilerplate as is possible. I (wrongly) assumed that "use webpack-dev-middleware" meant that more code would need to be added to the project.

I've since submitted a pull request that results in a net reduction in lines of code and also brings the overlay into the picture. You can check it out here: #81

from react-hot-boilerplate.