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.

Sign up for GitHub

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

Jump to bottom

Better Die handling #463

Merged

ghostdogpr merged 1 commit into master from better-die-handling

Jun 24, 2020

Owner

ghostdogpr commented Jun 23, 2020

Closes #462

Die case will be handled in Executor rather than adapters. This has 2 advantages:

die errors have path and locations attached
die errors can be handled by GraphQLInterpreter#mapError


          Better Die handling

de82af1

ghostdogpr mentioned this pull request

UIO / URIO field failures are not available in interpreter mapError #462

Closed

ghostdogpr merged commit 5400bbb into master

ghostdogpr deleted the better-die-handling branch

June 24, 2020 00:04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment