:npm-deps dependencies are implicit

Description

Currently the Node dependencies for `:npm-deps` to work are implicit. Looking `cljs/module_deps.js` it's clear that at least `module-deps`, `resolve`, and `browserResolve` must exist. This should probably be handled transparently for the user.

Environment

None

Activity

Show:
David Nolen
July 8, 2017, 6:09 PM

Even after manually installing all the necessary deps if I try to run just the `test-npm-deps` test by itself on master I get the following stacktrace from Node.js:

David Nolen
July 8, 2017, 3:52 PM

On my local machine at least I had to install these manually. I think this should probably be handled for the user?

Completed

Assignee

David Nolen

Reporter

David Nolen

Labels

Approval

Accepted

Patch

Code