Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

GraphQL Interpreter #62

Merged
merged 2 commits into from
Apr 20, 2020
Merged

GraphQL Interpreter #62

merged 2 commits into from
Apr 20, 2020

Conversation

exAspArk
Copy link
Owner

@exAspArk exAspArk commented Apr 8, 2020 •
edited
Loading

  • Fixes using BatchLoader.for with the GraphQL Interpreter Support for new GraphQL Interpreter #41
  • Deprecates using BatchLoader.for in favor of BatchLoader::GraphQL.for (or BatchLoader::GraphQL.wrap in resolvers)

@coveralls
Copy link

coveralls commented Apr 8, 2020 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling cd396ff on graphql-interpreter into 4b7596a on master.

@exAspArk exAspArk mentioned this pull request Apr 8, 2020
Closed
@exAspArk exAspArk force-pushed the graphql-interpreter branch from ac18160 to 4f9c547 Compare April 8, 2020 16:58
@exAspArk exAspArk force-pushed the graphql-interpreter branch from 4f9c547 to 7bb3eea Compare April 8, 2020 16:59
@exAspArk exAspArk force-pushed the graphql-interpreter branch from 7bb3eea to cd396ff Compare April 8, 2020 17:14
@exAspArk exAspArk marked this pull request as ready for review April 8, 2020 17:22
@exAspArk exAspArk merged commit 071f943 into master Apr 20, 2020
@exAspArk exAspArk deleted the graphql-interpreter branch April 20, 2020 18:16
@stanhu stanhu mentioned this pull request Feb 15, 2023
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@exAspArk @coveralls