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

Puya integration into algokit-cli, removing the need to install them explicitly #400

Open
3 of 4 tasks
aorumbayev opened this issue Jan 30, 2024 · 1 comment · Fixed by #441
Open
3 of 4 tasks
Assignees
Labels
algokitv3 AlgoKit v3 backlog items

Comments

@aorumbayev
Copy link
Collaborator

aorumbayev commented Jan 30, 2024

Description

The goal is to ensure that algokit-cli handles installation of puya-py automatically and lazy-loads them as per usage.

Technical Details

Using algokit compile uses approach similar to algokit generate where semantics are inferred based on file extension. .py -> puya. An alternate approach is to use algokit compile py, so the compiler is known when the command is executed. This also has the added benefit of be extensible for future support of other compilers that may require different arguments.

We want to integrate this into the cli once 1.0 of Puya has released and update the cli to v2.

Acceptance Criteria

  • A user is able to execute the puya compiler via algokit-cli and turn their PuyaPy smart contract into a deployable teal contract, without explicitly installing it by executing algokit compile *.py
  • A user is able to discover any compilation issues in their PuyaPy contract using algokit compile

Tasks

  • PuyaPy compiler is integrated into the cli, usage is tested
  • show the compile command and add documentation
@aorumbayev aorumbayev added this to the AlgoKit CLI enhancements milestone Jan 30, 2024
@aorumbayev aorumbayev added algokitv3 AlgoKit v3 backlog items feature labels Jan 30, 2024
@neilcampbell neilcampbell changed the title Puya & Tealscript integration into algokit-cli, removing the need to install them explicitly Puya integration into algokit-cli, removing the need to install them explicitly Feb 26, 2024
@PatrickDinh PatrickDinh self-assigned this Feb 28, 2024
@PatrickDinh
Copy link
Contributor

Reopen because this will be documented in v2 release.

@PatrickDinh PatrickDinh reopened this Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
algokitv3 AlgoKit v3 backlog items
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants