Skip to content

Remove ignite/starport cli from codebase#27

Merged
philipsu522 merged 7 commits intomasterfrom
rm-ignite-cli
Jun 3, 2022
Merged

Remove ignite/starport cli from codebase#27
philipsu522 merged 7 commits intomasterfrom
rm-ignite-cli

Conversation

@philipsu522
Copy link
Contributor

@philipsu522 philipsu522 commented May 31, 2022

Description

Remove ignite/starport CLI from our codebase so that we can set configs (config.toml, app.toml) at the sei code level (see cmd/seid/cmd/init.go. This will allow everyone running the chain to have unified configs that we can control and change with upgrades.

Testing

  • Verified that starport no longer exists:
 ~/sei-chain  rm-ignite-cli *1 ?1  seid version --long | grep star                                                     ok  21:04:40
 ~/sei-chain  rm-ignite-cli *1 ?1
  • Start and restart a chain with default chain name
  • Start and restart a chain with a unique name
  • Create account
  • List keys
  • Send funds from one account to another
  • Query validator set
  • Create a param change proposal
  • Voted on proposal
  • Verified proposal passed

@philipsu522 philipsu522 changed the title [WIP] Remove ignite/starport cli from codebase Remove ignite/starport cli from codebase Jun 1, 2022
@philipsu522 philipsu522 requested a review from faddat June 1, 2022 02:36
@philipsu522 philipsu522 requested review from LCyson and codchen June 1, 2022 02:36
@sei-protocol sei-protocol deleted a comment from icodragon Jun 2, 2022
@sei-protocol sei-protocol deleted a comment from icodragon Jun 2, 2022
@philipsu522 philipsu522 merged commit d436da8 into master Jun 3, 2022
masih pushed a commit that referenced this pull request Sep 29, 2025
* Add Tx channels to context

* comment

* use access operations instead

* fix with handlers

* Fixed dependency

* add message index

* alias

* alias

* msg index
masih pushed a commit that referenced this pull request Sep 30, 2025
* Add Tx channels to context

* comment

* use access operations instead

* fix with handlers

* Fixed dependency

* add message index

* alias

* alias

* msg index
masih pushed a commit that referenced this pull request Oct 9, 2025
@masih masih deleted the rm-ignite-cli branch October 31, 2025 15:51
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.

5 participants