-
Notifications
You must be signed in to change notification settings - Fork 34
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
feat: python support #282
feat: python support #282
Conversation
- checking master
- fixed function calls and generation of nested schemas - fixed runtime byte packing
While writing some tests I came over some issues with decoding in the runtime. So I converted the PR to draft for now. |
- array types needed obviously append instead of assignment - runtime had some read issues for bytes and guid - added unit tests
- writing enum was faulty, had to call value instead
Hi, currently recovering from the flu so I’ll review once I’m back on my feet. Two things:
|
Hey @andrewmd5 no worries! Regarding your inquiries,
|
You shouldn’t need to open a new PR. Just squash and force push to your branch. And yes, the roundtrip integration test suite will determine if your implementation is correct. |
Okay thanks will do that then :) |
Let me know if you need any help with the integration test suite |
solves #56