apollographql / apollo-tooling

✏️ Apollo CLI for client tooling (Mostly replaced by Rover)
https://apollographql.com
MIT License
3.04k stars 467 forks source link

apollo-language-server: Include graphql as a peer dependency #2652

Open HishamAli81 opened 1 year ago

HishamAli81 commented 1 year ago

Intended outcome:

Graphql to be included as a peer dependency in the apollo-language-server package, similar to how it's included in the apollo-graphql package (https://github.com/apollographql/apollo-tooling/blob/master/packages/apollo-graphql/package.json#L19).

Nice to have: Add support for GraphQL 16.

Actual outcome:

Graphql is currently being included as a dependency in the apollo-language-server package (https://github.com/apollographql/apollo-tooling/blob/master/packages/apollo-language-server/package.json#L37), and doesn't follow these guidelines: https://leebyron.com/graphql-js-preparing-for-v14/

Versions

apollo-language-server: 1.26.9