Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
upgradeclient "github.com/cosmos/cosmos-sdk/x/upgrade/client" | ||
) | ||
|
||
var ModuleBasics = []module.AppModuleBasic{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this for encodings or decoding txs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both
Debug: debug, | ||
Timeout: "20s", | ||
OutputFormat: "json", | ||
BroadcastMode: "block", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should avoid using block, this was meant for testing that ended up bleeding into prod.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh interesting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess I should add a mempool query that sees if we have a TX in the mempool before trying to submit another one
closing this in favor of other work. Come back to this once cobra merges spf13/cobra#1551 |
This PR will move the config parsing to stuffing the chains into the
context.Context
where they will be consumed in the cli functions