-
Notifications
You must be signed in to change notification settings - Fork 180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bad error catch #11
Comments
Hey, it looks like you need to install the server require hook. |
I have the same issue as ArvoGuo has suggested, if i run webpack-dev-server, all is okay. But if introduce a syntax error in a Js file to invalidate the build, the server then crashes instead of recovering gracefully
|
Oh, I missed the paren on the second line (as I never used jsx) and automatically assumed it is a Could you please provide your webpack config and a snippet that fails? I’ve been using css modules for a quite a while now and did not encounter this. And believe me when I say I make a lot of typos and syntax errors. However, I’m using coffee-script so this might be babel/jsx related? |
Hi, thanks for looking, was an untouched clone of this repo, so the config is https://github.com/css-modules/webpack-demo/blob/master/webpack.config.js Heres a gist of the error log: https://gist.github.com/alexFaunt/39dc35adbfc5608fe51d I'm a bit of a webpack beginner so i'm not sure what i'm looking at. I've been playing around using webpack --watch instead |
So, as I thought, the issue is related to What I did:
<html>
<head>
<title>CSS Modules Webpack Demo</title>
<meta name="viewport" content="width=device-width, initial-scale=1" />
<link rel="stylesheet" type="text/css" href="style.css" />
</head>
<body>
<div id="outlet"></div>
<script src="/index.js"></script>
</body>
</html> This should do it for now. I’ll fork this repo and edit it to be more dev-friendly. I’ll let you know once it’s done. |
Amazing, cheers |
No problem, @alexFaunt. I forked this repo, removed some unnecessary bits and dependencies and added gaearon's awesome react-hot-loader. It’s not ready for production for now, but I plan on keeping it up to date. You can check it out here: |
if there is a error of syntax like
the server just be broken!
The text was updated successfully, but these errors were encountered: