Skip to content
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

Missing webpack-stats on setup? #43

Closed
nathanmsmith opened this issue May 31, 2018 · 6 comments · Fixed by #42
Closed

Missing webpack-stats on setup? #43

nathanmsmith opened this issue May 31, 2018 · 6 comments · Fixed by #42

Comments

@nathanmsmith
Copy link
Contributor

nathanmsmith commented May 31, 2018

Tried setting up Kerckhoff for the first time and I got this error after running docker-compose up:
screen shot 2018-05-31 at 10 11 25 am

I noticed that webpack-stats.json is in both .gitignore and .dockerignore. I'm not familiar with it, but I would assume that it needs to be autogenerated by webpack? Why then was it not autogenerated on my machine during setup? I'm guessing what's happening is it's looking for webpack-stats.json before webpack runs?

@hongee
Copy link
Contributor

hongee commented May 31, 2018

we changed the way we build frontend on dev cos its causing a lot of problems between osx/windows - so this is no longer automated but instead you'll need to run npm run watch locally to build the package dependencies. For production it is still built automatically

@nathanmsmith
Copy link
Contributor Author

So run npm run watch before docker-compose up?

@hongee
Copy link
Contributor

hongee commented May 31, 2018

you can run it whenever - once that's ran the error will disappear

@nathanmsmith
Copy link
Contributor Author

Do the node_modules need to be installed locally also, i.e., do I need to run npm install?

@hongee
Copy link
Contributor

hongee commented May 31, 2018

yeap

@nathanmsmith
Copy link
Contributor Author

nathanmsmith commented May 31, 2018

aite added documentation in #42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants