-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Update types against GraphQL 0.8.2 and @types/graphql #934
Conversation
Also depends on this: DefinitelyTyped/DefinitelyTyped#12800 to fix GraphQLError usage in tests |
a1e60a8
to
a05f116
Compare
That fixes all of the errors throughout compilation. I'm still getting errors about Namespace |
Also requires: https://github.com/apollostack/graphql-anywhere/pull/16 |
a05f116
to
c558cf4
Compare
all depended branches were merged. This should be ready to go |
@intellix Awesome! Do you have time to rebase/merge the branch? As soon as that's done, we should be ready to drop typed-graphql. |
af153b5
to
3dc1f3d
Compare
Your wish is my command! have fun re-reviewing :P |
Closing in favor of #1041 |
TODO:
closes #927
depends on: DefinitelyTyped/DefinitelyTyped#12767
Currently getting a few errors with GraphQLError type as the @types/graphql is saying some fields are required. Some changes are needed before that gets merged and then this can be.
Just wanted to push some work before bed and someone else spends the same time on this :)