-
Notifications
You must be signed in to change notification settings - Fork 2.2k
HOST=[host] NODE_ENV=production npm run depoy gives error #571
Comments
HOST=[host] NODE_ENV=production npm run depoy
gives error
Could you please post your |
I haven't been able to reproduce this, do you have any additional information about this error? |
Sorry for my late replay, it is here: |
Exactly the same error on my side. |
@Donovan-Ensicaen The most weird thing to me is that this is a transitional issue. It never happened in my desktop, but it bothers me on the server side SOMETIMES, all with the same code base, running the same command, which is |
@hophacker I use |
Ok it works on my side, I can run |
Having the same issue here. Not able to deploy on heroku. |
It seems that you need to clear all the errors output showing on "npm run On Mon, Feb 29, 2016 at 1:33 PM, Tony Tai Nguyen [email protected]
冯杰 Cell: 86-15862355582 Prev: Comprehend Systems, 2010 Broadway #200, Redwood City, CA 94063 |
I might try deploying it myself to heroku today to see if I can repro. |
Thanks @davezuko. I tried to just git pull / build / deploy and it was not working on my side. |
I have the same issue when I try to deploy on heroku or a private VPS server on CentOS. Locally (OSX El Capitan) both Same |
Here is the excerpt from my
My app is based on this (awesome! :) ) starter-kit and all the main important config files left almost untouched. |
I figured out what caused this error. I did After that, when I ran However I had to add // ...
// ------------------------------------
// Apply Webpack HMR Middleware
// ------------------------------------
if (config.env === 'development') {
const webpackDevMiddleware = require('./middleware/webpack-dev').default
const webpackHMRMiddleware = require('./middleware/webpack-hmr').default
const compiler = webpack(webpackConfig)
// Enable webpack-dev and webpack-hot middleware
const { publicPath } = webpackConfig.output
app.use(webpackDevMiddleware(compiler, publicPath))
app.use(webpackHMRMiddleware(compiler))
// ... |
Thanks for the info @romanenko, I will make those changes in master later. |
Oh thanks @romanenko ! Not easy to catch. I hope I will be able to deploy on heroku after this fix :) |
Going to close this for now since the issue is actually tangential as @romanenko has figured out (production-only dependencies) and has a bigger scope (improved error reporting). I will be creating a more formal issue to tackle this once I return from vacation. |
The text was updated successfully, but these errors were encountered: