Postgraphile: graphql dependency

[lifeiscontent]

I'm submitting a ...

• [x] bug report
• [ ] feature request
• [ ] question

PostGraphile version: 4.0.0-beta.3

Minimal SQL file that can be loaded into a clean database:

Steps to reproduce:

Setup react-apollo project.

Current behavior:

the graphql dependency that is in postgraphile is conflicting with the app I'm writing because it's pretty outdated.

Expected behavior:

would like to see the dependency updated so I can continue working on my project.

[benjie]

We support "graphql": ">=0.6 <0.13" because we can't go higher due to codemirror-graphql for GraphiQL; please forward your concerns here:

https://github.com/graphql/codemirror-graphql

[barbalex]

done: https://github.com/graphql/codemirror-graphql/issues/247

[benjie]

Wow; I recommend using yarn - that should clear up a lot of those duplicates for you. Add [email protected] to package.json or use the yarn resolutions feature.

[barbalex]

@benjie

I _do_ use yarn. And the resolutions feature.

But I messed up by updating the graphql version in the dependencies key to v13.2 while leaving its version in the resolution key at 13.0 :-(

Only noticed thanks to your comment. Thanks!

[benjie]

Ahhhh; that explains it!

[james-ff]

@barbalex Hi, are you able to share your yarn resolutions fix?

[barbalex]

@james-ff : this is my package.json file: https://github.com/barbalex/apf2/blob/34f75b9604389e4b9275811d46c3112be3d2b30f/package.json#L94-L96

The treacherous part of this fix is when you update graphql itself: don't forget to also update the resolution!

[james-ff]

Seems that the library in question now supports up to v14 of GraphQL =]

https://github.com/graphql/codemirror-graphql/commit/a4a0ff558622247e8d7ccc0acf978f9e0078dec7

[benjie]

https://github.com/graphql/graphiql/blob/master/package.json

GraphiQL doesn’t yet though 😔

[james-ff]

There's always one =P.

I rest hope in this PR https://github.com/graphql/graphiql/pull/728.