Skip to content

Add update-contract command#6

Merged
psiemens merged 2 commits into
masterfrom
3-update-contract
Dec 1, 2020
Merged

Add update-contract command#6
psiemens merged 2 commits into
masterfrom
3-update-contract

Conversation

@psiemens
Copy link
Copy Markdown
Contributor

Closes #3

Description

This PR adds the flow accounts update-contract command, which allows users to update a contract deployed to an account.

This new command makes use of the templates.UpdateAccountContract template from the latest release of the Flow Go SDK.

Misc. changes:

  • Updated flag capitalization to be consistent across all commands.

For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@psiemens psiemens requested a review from Kay-Zee as a code owner November 30, 2020 01:26
@psiemens psiemens requested a review from turbolent November 30, 2020 01:26
Copy link
Copy Markdown
Member

@turbolent turbolent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Copy link
Copy Markdown
Member

@turbolent turbolent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice to duplicate this as add-contract and using AddAccountContract

@psiemens psiemens added the Feature A new user feature or a new package API label Dec 1, 2020
@psiemens psiemens merged commit a80910f into master Dec 1, 2020
@psiemens psiemens deleted the 3-update-contract branch December 1, 2020 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature A new user feature or a new package API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for upgrading contracts on an account

3 participants