-
Notifications
You must be signed in to change notification settings - Fork 43
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
Install npm dependencies automatically #28
Comments
What command are you running that you expect the hook to run on? It should already run I'm not confident in this behaviour working perfectly, although I believe that is the intention. |
After looking into this some more it seems like we may be able to insert |
This hook exists right now, if you run |
Thanks for looking into this. I have issues upgrading to 0.6.0, it looks like
|
Jumping a bit through the leiningen code it looks like dependencies are loaded via lein-npm seems to hook into |
Yeah, the only invocation of When I changed the hook to be on |
I am having this problem. Heroku run lein uberjar to build the project and the node modules are not being fetched. Running lein uberjar on my own machine has the same effect. |
@pupeno coudn't you just run |
I don't need |
Is there a way to configure lein-npm to automagically install npm dependencies when running
lein
?As stated in the "Clean code" book, there should be only one command to run the build. It would be nice if projects could include npm deps in their bootstrapping.
The text was updated successfully, but these errors were encountered: