mst-gql icon indicating copy to clipboard operation
mst-gql copied to clipboard

Mobx 6 and MST 5 support?

Open arodik opened this issue 4 years ago • 3 comments

Hi. I want to use this fancy library, but it turns out that it requires old MST and Mobx in the peer dependencies list. Dev dependencies list requires fresher versions, so I'm confused a little bit. Do you have plans to support MST 5 and Mobx 6?

arodik avatar Jun 02 '21 10:06 arodik

Will take care of this, thanking you for pointing it out. The library should support whatever is listed in the dev dependencies list @arodik.

jesse-savary avatar Jun 10 '21 01:06 jesse-savary

Are these peerDependencies really needed? It trips rush package management.

 ERROR  @rush-temp/fe-models: [email protected] requires a peer of mobx@^4.0.0 || ^5.0.0 but version 6.3.3 was installed.

smokku avatar Sep 01 '21 12:09 smokku

I appreciate that npm vs yarn is a crazily opinionated subject but sometimes you have to use npm because of external reaons!

I found I had to use specific versions and then force npm to ignore the mismatches:

$ npm install [email protected] [email protected] [email protected] [email protected] graphql-request
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: [email protected]
npm ERR! Found: [email protected]
npm ERR! node_modules/mobx
npm ERR!   mobx@"6.1.8" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer mobx@"^4.0.0 || ^5.0.0" from [email protected]
npm ERR! node_modules/mst-gql
npm ERR!   mst-gql@"0.14.1" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!

I read over all the peer package dependencies, nothing jumps out at me as an incompatibility and all these versions should work together (so ran npm install [email protected] [email protected] [email protected] [email protected] graphql-request --force)

SacredSkull avatar Dec 01 '21 12:12 SacredSkull

Should be resolved as of 0.17.

jesse-savary avatar Nov 06 '22 20:11 jesse-savary