Pavel Nikolov
Pavel Nikolov
Closing this PR due to inactivity and since #400 takes care of it.
I'm closing this issue as this is part of the spec requirements. @VincentVW if you think that the lib is violating the spec for lists I'd be happy to reopen...
@romshark I'm closing this issue due to inactivity. I'm not able to reproduce it locally.
[ValidateWithVariables](https://pkg.go.dev/github.com/graph-gophers/graphql-go#Schema.ValidateWithVariables) is implemented now.
This issue has been fixed by #364 Now you would get an error: ``` root operation "Query" must be defined ``` This is [the code that returns the error](https://github.com/graph-gophers/graphql-go/pull/364/files#diff-98bf38fb290ecb143188e1585622022aaa0711a888da5e74b254e01a66d0eb99R257).
I'm closing this issue as it doesn't currently solve some issue with this library.
@abourget I think that #420 solve this issue, right? And I believe that [this unit test demonstrates the ability to propagate subscription errors to the user](https://github.com/graph-gophers/graphql-go/blob/master/subscription_test.go#L123-L168). Let me know if...
I'm closing this PR due to inactivity and I believe that it is no longer relevant after the tracing refactoring.
Closing this due to inactivity. I'm also adding a lot of executable examples and improving documentation in the next release.
This change is something we need as we are hitting network congestion limits too in many of our m3coordinator pods. @jorgelbg I believe that the errors that you are facing...