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

Convert to using the Elixir thrift parser from pinterest/elixir-thrift #11

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

dantswain
Copy link
Owner

https://github.com/pinterest/elixir-thrift#thrift-idl-parsing

This would be great to use because it avoids needing to have the thrift binary installed, and it means working directly with Elixir types rather than translating from the thrift-generated Erlang code. It also makes it possible to resolve #10 which I could not resolve using the Erlang route.

The code is currently really rough and needs refactoring badly, but the proof of concept is there. I intend to refactor it heavily.

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.

Constant structs do not work.
1 participant