appsforartists / ambidex-example--bike-index

An example demonstrating how to utilize Ambidex.
16 stars 6 forks source link

The package react-tap-event-plugin does not satisfy its siblings' peerDependencies requirements! #9

Open vespakoen opened 9 years ago

vespakoen commented 9 years ago

Hey!

I tried installing this example app. Unfortunately, there are some dependency conflicts. Tried with 0.10, 0.11 and 0.12 (using nvm)

Here is the last error I got (with 0.12)

npm WARN package.json make-node-env-global@1.0.0 No README data
npm ERR! Linux 3.16.0-34-generic
npm ERR! argv "node" "/home/koen/.nvm/versions/node/v0.12.2/bin/npm" "install"
npm ERR! node v0.12.2
npm ERR! npm  v2.8.3
npm ERR! code EPEERINVALID

npm ERR! peerinvalid The package react-tap-event-plugin does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer ambidex@0.2.0-beta4 wants react-tap-event-plugin@0.1.3
npm ERR! peerinvalid Peer gravel@0.0.2 wants react-tap-event-plugin@~0.1.3

Any ideas?

appsforartists commented 9 years ago

That sounds like a bug in npm. As I understand it, 0.1.3 and ~0.1.3 should intersect on 0.1.3.

I just made sure there are explicit dependencies for all the inherited peerDependencies ( in preparation for https://github.com/npm/npm/issues/6565).

Try this:

git pull origin master
rm -rf node_modules
npm install
rm -rf node_modules/node-jsx/node_modules/react-tools/
rm -rf node_modules/ambidex/node_modules/jsx-loader/node_modules/react-tools/

That will make sure you don't have any old versions of react-tools lying around. I really should create a PR for jsx-loader to upgrade to react-tools@0.13.2, but I've been lazy because react-tools altogether will be replaced with jstransform, and I'd rather not babysit a PR that could be obsolete by the time it lands.