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

feat: Automate deployments through tasks! #67

Merged
merged 9 commits into from
Jun 10, 2022
Merged

Conversation

octalmage
Copy link
Contributor

This PR includes a few features/improvements.

  1. The ability to call storeCode and instantiate from inside of tasks. I've added an example to the readme which can be viewed here: https://github.com/terra-money/terrain/blob/de5627172ec60da427c7abd5ae6721fa60051087/README.md#creating-tasks
  2. Moving reused flags to src/lib/flag.ts. There were situations where flags are reused, but the description or default was different depending on the command. In these situations we can move the flag to this new file, import it, and reference the same default value and description.
  3. Added a default value for --signer. When using LocalTerra, we can assume the signer will be the test1 wallet. This improves usability for local development.
  4. I ran the readme generator which overwrote some of the work by @terran6. If we do want to modify the oclif readme template we should do without manual tweaks to the readme, because they get reverted.
  5. Small tweaks and cleanup based on eslint rules.

@octalmage octalmage force-pushed the feat/deploy-tasks branch from dc6c5fe to d6163ec Compare May 6, 2022 17:34
src/config.ts Show resolved Hide resolved
src/config.ts Outdated Show resolved Hide resolved
src/config.ts Outdated Show resolved Hide resolved
@octalmage octalmage force-pushed the feat/deploy-tasks branch from e30e926 to 256db17 Compare June 10, 2022 02:26
@octalmage octalmage merged commit 765a9b7 into main Jun 10, 2022
@octalmage octalmage deleted the feat/deploy-tasks branch June 10, 2022 02:31
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

Successfully merging this pull request may close these issues.

3 participants