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 gqlparser to v2.5.0 #2341

Merged
merged 1 commit into from
Aug 26, 2022
Merged

Update gqlparser to v2.5.0 #2341

merged 1 commit into from
Aug 26, 2022

Conversation

StevenACoffman
Copy link
Collaborator

@StevenACoffman StevenACoffman commented Aug 26, 2022

Per vektah/gqlparser#234 and vektah/gqlparser#99 and vektah/gqlparser#100

In gqlparser all methods which were returning an error were using the explicit struct type (e.g. *gqlerror.Error or gqlerror.List). As long as you used the explicit types everything was fine, but as soon as you converted them to the error interface type, nil checks would fail. As a result, gqlparser recently was changed to return error interface, so we need to perform a type assertion to convert it to *gqlerror.Error in some places. It seems to me like this is a fairly minor change for those using gqlgen directly. What do you think?

The current gqlgen release version is v0.17.15. If this is deemed enough of a breaking change, we can release this as v0.18.0,
otherwise, I will proceed with v0.17.16

Signed-off-by: Steve Coffman steve@khanacademy.org

Signed-off-by: Steve Coffman <steve@khanacademy.org>
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 75.428% when pulling 01bec3a on update_gqlparser_v2_5_0 into 2a87fe0 on master.

@StevenACoffman StevenACoffman merged commit 2390af2 into master Aug 26, 2022
@StevenACoffman StevenACoffman deleted the update_gqlparser_v2_5_0 branch August 26, 2022 20:16
StevenACoffman added a commit that referenced this pull request Sep 13, 2022
Signed-off-by: Steve Coffman <steve@khanacademy.org>
StevenACoffman added a commit that referenced this pull request Sep 13, 2022
* nil check error before cast from #2341

Signed-off-by: Steve Coffman <steve@khanacademy.org>

* Improve errcode.Set safety

Signed-off-by: Steve Coffman <steve@khanacademy.org>

Signed-off-by: Steve Coffman <steve@khanacademy.org>
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.

2 participants