-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Installing app dependencies #326
Comments
Works for me.
No. |
Why? Your app directory is app, so, run your app in this dir. Why in the root? E.g. |
In your example project you compile the |
Got your problem. So, solution — use NODE_PATH env var to add `app/node_modules' to node search path in development mode. We use this approach in our tests, for example (https://github.com/electron-userland/electron-builder/blob/master/.idea/runConfigurations/osxPackagerTest.xml#L4). |
My final approach: https://github.com/sqlectron/sqlectron-gui/blob/master/src/browser/main.js#L12. Thanks @develar. Do you think it should be documented some where or it is too specific to my implementation? |
Your workflow is very common. Should be documented. |
@develar are you going to work on this? I can document it if you need any help. |
@maxcnunes Please document, help is wanted ;) |
Wow awesome blog post. Really thank you! Fantastic.1000pip builder signals Review |
I am following the suggested folder structure for the project with 2
package.json
.The
/app/package.json
includes only the dependencies for the release version and the/package.json
(root folder) includes the dependencies for the development environment.Although even with the
"postinstall": "install-app-deps"
in the/package.json
. The npm install does not install the dependencies from the/app/package.json
.Do I have to duplicate the dependencies from
/app/package.json
in the/package.json
? Would be good if theinstall-app-deps
could read the/app/package.json
and install it automatically.The text was updated successfully, but these errors were encountered: