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

osmocli epic: #3641

Closed
6 of 7 tasks
ValarDragon opened this issue Dec 6, 2022 · 1 comment
Closed
6 of 7 tasks

osmocli epic: #3641

ValarDragon opened this issue Dec 6, 2022 · 1 comment

Comments

@ValarDragon
Copy link
Member

ValarDragon commented Dec 6, 2022

OsmoCli (#3611) roadmap of things to tackle to improve testing UX:

  • Add the ability to register a custom parser for a given field. (Lets us eliminate the rest of existing boilerplate / non-converted funcs)
  • Add some go tests within osmo-cli
  • Change the syntax from cmd.AddCommands to instead be osmocli.AddTx[M sdk.Msg](&TxDesc, msg M)
    • Need to figure out how to register custom flagsets in this syntax
    • Lets us make a second method, for getting a proto-parser from desc, and then we can kill all the current slow CLI tests
  • README
  • Split this into its own go.mod, so others can import
@pysel
Copy link
Member

pysel commented Dec 6, 2022

hello @ValarDragon. I would like to collaborate here, is it ok with you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

2 participants