-
Notifications
You must be signed in to change notification settings - Fork 48
npm install fails #47
Comments
@phaneendra that's unfortunate. |
@phaneendra This should no longer be necessary as of 3.0.0-rc7. Could you please verify without babel dependencies? |
Yes rc7 works with Now
|
@phaneendra that's an npm issue. quick solution is to either update your npm to 3 ( Because you're using Node 4.2.2's shipped version of npm (npm v2.14.7), it's not able to inspect the full dependency tree. If you want to stay with npm@2 and use universal-redux you'll need to include many of UR's dependencies as dependencies of your own project. It's too unwieldy to track what those are since I've moved on from npm 2, but it looks like in this case you'd need to at least bring all the babel stuff over. I strongly recommend going with npm3. It can be a bit slower, but you'll probably run into this issue with other libraries as well. With the |
upgrading to npm 3 and it flat module resolution did the trick, I feel explicitly depending on universal redux dependencies defeats the purpose of why this module was converted to npm module. |
@phaneendra agreed. i've decided to make npm3 a requirement now. it will provide a warning on npm install otherwise. |
npm install universal-redux fails with no babel command found
Is it required to install babel-core beforehand
The text was updated successfully, but these errors were encountered: