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

remove Query enum #37

Open
tromey opened this issue Dec 24, 2017 · 1 comment
Open

remove Query enum #37

tromey opened this issue Dec 24, 2017 · 1 comment

Comments

@tromey
Copy link
Collaborator

tromey commented Dec 24, 2017

I don't think there's a reason to have Query be a separate enum. They are really just commands.

@luser
Copy link
Owner

luser commented Dec 27, 2017

I think the only thing I was aiming for there was having the query parser be able to return only Query values, but it might not really provide any extra value, and practically everything is a query in this protocol anyway...

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

No branches or pull requests

2 participants