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

pop build contract: add default contracts directory #42

Closed
brunopgalvao opened this issue Mar 8, 2024 · 4 comments
Closed

pop build contract: add default contracts directory #42

brunopgalvao opened this issue Mar 8, 2024 · 4 comments

Comments

@brunopgalvao
Copy link
Contributor

As a dev, I would like to have a seamless UX flow for building contracts in which I created with Pop, so that I don't have to waste time specifying configuration that should already by part of the Pop flow.

The UX flow for building contracts with Pop should be the following:

pop new contract my_contract
pop build contract my_contract

Currently it is not so, currently this is the flow:

pop new contract my_contract
pop build contract -p ./my_contract

-p should be an optional argument when the developer is storing their contracts in another directory other than the default.

@Daanvdplas
Copy link
Collaborator

I would even add to this that when the dev is in the repo of the contract it should be able to run pop build and pop cli will detect that it is in a contract folder and run cargo contract build under the hood.

Detecting whether the dev wants to execute a pop cli command on parachain / contract folder will be useful in general.

@weezy20
Copy link
Contributor

weezy20 commented Apr 5, 2024

pop build contract my_contract can be integrated into #92.

@brunopgalvao
Copy link
Contributor Author

I would even add to this that when the dev is in the repo of the contract it should be able to run pop build and pop cli will detect that it is in a contract folder and run cargo contract build under the hood.

Detecting whether the dev wants to execute a pop cli command on parachain / contract folder will be useful in general.

whao I like this pop build

@AlexD10S
Copy link
Collaborator

Closing this Issue as it will be solved with this one: #91

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

4 participants