Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Independence #529

Closed
wants to merge 9 commits into from
Closed

Independence #529

wants to merge 9 commits into from

Conversation

kamilkisiela
Copy link
Collaborator

@kamilkisiela kamilkisiela commented Mar 30, 2020

  • no longer depends on graphql-tools
  • breaks logic into few packages
  • new package @graphql-toolkit/types - contains only types like IResolvers etc
  • new package @graphql-toolkit/version - checks the version of GraphQL (v14+ exports versionInfo)
  • new package @graphql-toolkit/schema - has makeExecutableSchema (+remote) and other related functions
  • should reduce the bundle size
  • use range for tslib

@theguild-bot
Copy link

theguild-bot commented Mar 30, 2020

The latest changes of this PR are available as alpha in npm: 0.10.3-alpha-0a56f31.6+0a56f31

Quickly update your package.json by running:

npx match-version @graphql-toolkit 0.10.3-alpha-0a56f31.6+0a56f31

@kamilkisiela kamilkisiela force-pushed the share-tools branch 3 times, most recently from a4109a8 to 6d021cc Compare April 6, 2020 06:35
@kamilkisiela kamilkisiela changed the title Share tools Independence Apr 6, 2020
definitions: [info.operation, ...fragments],
};

return fetcher({
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will cause some issues later. Let’s say remote schema is merged with another schema. It will send the whole query to the remote schema as well so some weird errors might be thrown. And also this will send the query for each root field.
ardatan/graphql-tools#1346 (comment)
This issue looks similar but it is for links. (Links work in a different way)

@ardatan ardatan closed this Apr 24, 2020
@ardatan ardatan deleted the share-tools branch April 24, 2020 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants