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

Update upstream v1.5.0 #6

Merged
merged 31 commits into from
Feb 13, 2024
Merged

Update upstream v1.5.0 #6

merged 31 commits into from
Feb 13, 2024

Conversation

Anufant
Copy link

@Anufant Anufant commented Feb 8, 2024

Update upstream graphql-go to v1.5.0

pavelnikolov and others added 30 commits January 20, 2022 10:09
Add OpenTelemetry tracer implementation
Add basic support for Apollo Federation

Co-authored-by: Alam <[email protected]>
Co-authored-by: pavelnikolov <[email protected]>
add support for repeatable directives
The `Friends` field had higher priority than the `FriendsResolver` method. This is the reason why the field was renamed to a value, that doesn't match the GraphQL resolver.
Remove dependency for graphql-go on OpenTracing and OpenTelemetry except
 where those tracers are explicitly configured for use.
…ph-gophers#516)

Improve type assertion method argument validation (require zero)

It's tempting to include a context argument (or think it's allowed), but
not discover that this will fail until a query is executed. Validating
the resolver during schema parsing reduces the chance of inadvertant
errors here.

Signed-off-by: Evan Owen <[email protected]>
* Disallow repeat of non repeatable directives

* Remove unnecessary scallar
fix minor unreachable code caused by t.Fatalf
* Fix __type queries sometimes not returning data

If the queried type was not defined it would return an empty data
section instead of `__type: null`

Fixes graph-gophers#539
@Anufant Anufant self-assigned this Feb 8, 2024
@Anufant Anufant merged commit e4f6130 into master Feb 13, 2024
3 checks passed
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.