8000 fix(hotfix): v0.17.2 by ilgooz · Pull Request #1448 · ignite/cli · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix(hotfix): v0.17.2 #1448

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

Merged
merged 7 commits into from
Aug 17, 2021
Merged

fix(hotfix): v0.17.2 #1448

merged 7 commits into from
Aug 17, 2021

Conversation

ilgooz
Copy link
Member
@ilgooz ilgooz commented Aug 10, 2021

No description provided.

* feat(templates/cmd): upgrade spm to add 'config' cmd

to appd.

* also make $HOME/config/client.toml configurable through 'init.client'
section of chain's Starport config.yml.

* default to test

* fix

* fix

* updated changelog

* fix config application order

* fix

Co-authored-by: Denis Fadeev <denis@fadeev.org>
@ilgooz ilgooz changed the title feat(templates/cmd): upgrade spm to add 'config' cmd (#1417) hotfix: 0.17.2 Aug 10, 2021
@ilgooz ilgo 8000 oz changed the title hotfix: 0.17.2 fix: 0.17.2 Aug 10, 2021
ilgooz added 3 commits August 12, 2021 14:28
* fix(services/chain): detecting keyring

when client.toml > keyring-backend used.

* fix

* fix

* default to test kb

* fix
@ilgooz ilgooz marked this pull request as ready for review August 13, 2021 06:48
@ilgooz ilgooz changed the title fix: 0.17.2 fix(hotfix): v0.17.2 Aug 13, 2021
Copy link
Contributor
@fadeev fadeev left a comment

Choose a reason for hiding this comment

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

Stuff that works:

  • v0.43, tested with Umee
  • chains with dashes in name
  • keyring test/os
  • serving a chain scaffolded with v0.17.0

What doesn't work:

Following the IBC basic tutorial.

~/planet % rm -rf ~/.starport/relayer 
~/planet % starport relayer configure -a \
--source-rpc "http://0.0.0.0:26657" \
--source-faucet "http://0.0.0.0:4500" \
--source-port "blog" \
--source-version "blog-1" \
--source-gasprice "0.0000025stake" \
--source-prefix "cosmos" \
--source-gaslimit 300000 \
--target-rpc "http://0.0.0.0:26659" \
--target-faucet "http://0.0.0.0:4501" \
--target-port "blog" \
--target-version "blog-1" \
--target-gasprice "0.0000025stake" \
--target-prefix "cosmos" \
--target-gaslimit 300000
---------------------------------------------
Setting up chains
---------------------------------------------


🔐  Account on "source" is "cosmos19w8gqdgsvmkwek4szl0748x0eaw00a0lnn0pyj"
 
 |· faucet is not operational: invalid character 'c' looking for beginning of value
 |· (balance: -)

🔐  Account on "target" is "cosmos19w8gqdgsvmkwek4szl0748x0eaw00a0lnn0pyj"
 
 |· faucet is not operational: invalid character 'c' looking for beginning of value
 |· (balance: -)

⛓  Configured chains: earth-mars

Note: mnemonics for relayer accounts are stored in /Users/fadeev/.starport/relayer/config.yml unencrypted.
This may change in the future. Until then, use them only for small amounts of tokens.

@fadeev
Copy link
Contributor
fadeev commented Aug 13, 2021

Using the faucet command,

~/planet % starport chain faucet cosmos19w8gqdgsvmkwek4szl0748x0eaw00a0lnn0pyj 5token,1000stake
invalid character 'c' looking for beginning of value

* feat(pkg/chaincmd/runner): support YAML encoded CLI response

* Apply suggestions from code review

* fix

* fix

* fix

* fix
@ilgooz ilgooz merged commit d534217 into master Aug 17, 2021
@ilgooz ilgooz deleted the hotfix/0.17.2 branch August 17, 2021 09:50
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
* feat(templates/cmd): upgrade spm to add 'config' cmd (ignite#1417)

* feat(templates/cmd): upgrade spm to add 'config' cmd

to appd.

* also make $HOME/config/client.toml configurable through 'init.client'
section of chain's Starport config.yml.

* default to test

* fix

* fix

* updated changelog

* fix config application order

* fix

Co-authored-by: Denis Fadeev <denis@fadeev.org>

* fix(services/chain): detecting keyring (ignite#1449)

* fix(services/chain): detecting keyring

when client.toml > keyring-backend used.

* fix

* fix

* default to test kb

* fix

* fix(services/chain): default home dir for app (ignite#1457)

* fix(services/chain): support 0.43.x based chains (ignite#1455)

* Apply suggestions from code review

* Apply suggestions from code review

* feat(pkg/chaincmd/runner): support YAML encoded CLI response (ignite#1468)

* feat(pkg/chaincmd/runner): support YAML encoded CLI response

* Apply suggestions from code review

* fix

* fix

* fix

* fix

Co-authored-by: Denis Fadeev <denis@fadeev.org>
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.

2 participants
0