search

dangcuuson / graphql-schema-typescript

Generate TypeScript from GraphQL's schema type definitions
191 stars 36 forks source link

`graphql` on peerDependencies should be >=15? #60

Open robertu7 opened 3 years ago

robertu7 commented 3 years ago

As title, graphql@15 was in devDependencies but graphql@14 in peerDependencies, it makes npm install fails with ERESOLVE unable to resolve dependency tree error.

fmoga commented 3 years ago

I'm affected by this too

tony commented 3 years ago

Created a PR at #61. Looking good so far

However, I'm not sure if GraphQL 14 will still work. In fact, it may be more convenient to deprecate it and keep graphql-14 bug fixes in a branch, e.g. start 1.6. Though it's up to @dangcuuson

You can test it at @peergrade/graphql-schema-typescript