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

Fix From<tungstenite::Error> for non async_gremlin features #198

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

xD3CODER
Copy link
Contributor

@xD3CODER xD3CODER commented Oct 15, 2023

Fixes the trait From<tungstenite::Error> is not implemented for GremlinError with non async features.

I'm not sure this is the right way to solve the problem.

@xD3CODER xD3CODER force-pushed the fix/errors-tungstenite branch from 633bdf6 to 751486d Compare October 15, 2023 16:31
@xD3CODER xD3CODER marked this pull request as ready for review October 15, 2023 16:31
@xD3CODER xD3CODER force-pushed the fix/errors-tungstenite branch from 751486d to e97518d Compare October 19, 2023 12:45
@xD3CODER xD3CODER changed the title Fix From<tungstenite::Error> Fix From<tungstenite::Error> for non async_gremlin features Oct 19, 2023
@xD3CODER xD3CODER force-pushed the fix/errors-tungstenite branch from e97518d to 5cb5cd1 Compare October 19, 2023 12:48
@xD3CODER xD3CODER force-pushed the fix/errors-tungstenite branch from 5cb5cd1 to 42ae6b7 Compare October 19, 2023 12:52
@wolf4ood wolf4ood self-requested a review October 19, 2023 13:07
Copy link
Owner

@wolf4ood wolf4ood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @xD3CODER for this PR

lgtm!

@wolf4ood wolf4ood merged commit b1c8902 into wolf4ood:master Oct 19, 2023
4 checks passed
@github-actions github-actions bot mentioned this pull request Oct 19, 2023
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