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.

Sign up for GitHub

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

Jump to bottom

AutoCLI Sane Kebab Case #13499

Closed
Tracked by #11775
aaronc opened this issue Oct 10, 2022 · 0 comments · Fixed by #14316
Closed
Tracked by #11775

AutoCLI Sane Kebab Case #13499

aaronc opened this issue Oct 10, 2022 · 0 comments · Fixed by #14316
Assignees
@julienrbrt
Labels
C:CLI

Comments

@aaronc
Copy link
Member

aaronc commented Oct 10, 2022 •
edited
Loading

As discussed in #13251 (comment), the kebab case output from github.com/iancoleman/strcase does weird things with numbers like i-32 and bech-32. We should write a custom kebab case function for autocli that creates sane kebab case when there are numbers (ex. i32 instead of i-32).
This was referenced Oct 10, 2022
Closed
Merged
@julienrbrt julienrbrt self-assigned this Dec 14, 2022
@julienrbrt julienrbrt moved this to 📝 Todo in Cosmos-SDK Dec 14, 2022
@julienrbrt julienrbrt moved this from 📝 Todo to 💪 In Progress in Cosmos-SDK Dec 14, 2022
@julienrbrt julienrbrt mentioned this issue Dec 15, 2022
Merged
19 tasks
@julienrbrt julienrbrt moved this from 💪 In Progress to 👀 Needs Review in Cosmos-SDK Dec 15, 2022
Repository owner moved this from 👀 Needs Review to 👏 Done in Cosmos-SDK Dec 15, 2022
@tac0turtle tac0turtle removed this from Cosmos-SDK Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@julienrbrt julienrbrt

Labels
C:CLI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Add AutoCLI Sane Kebab Case cosmos/cosmos-sdk
3 participants
@aaronc @tac0turtle @julienrbrt