-
Notifications
You must be signed in to change notification settings - Fork 47
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: add compile py command #441
Conversation
- Compile Python to TEAL with Puyapy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @PatrickDinh , left some comments - also to clarify docs are removed temporarily? The plan is to rewrite it again once this goes public? Wonder if we should have the docs written beforehand and stashed somewhere internally to not spend time rewriting it once we make it public.
Thanks @aorumbayev for reviewing, good call out about the doc, I'll stash it locally. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Might be worth just confirming internally if we want algokit compile py
or algokit compile python
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PatrickDinh just minor comments left regarding my initial suggestions with tests, lmk what you think
Proposed Changes
Fixes #400