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

[feature request] customizable subcommand separator besides space #1420

Open
samuela opened this issue Feb 19, 2019 · 5 comments
Open

[feature request] customizable subcommand separator besides space #1420

samuela opened this issue Feb 19, 2019 · 5 comments
Labels
A-builder Area: Builder API A-parsing Area: Parser's logic and needs it changed somehow. C-enhancement Category: Raise on the bar on expectations S-waiting-on-design Status: Waiting on user-facing design to be resolved before implementing

Comments

@samuela
Copy link

samuela commented Feb 19, 2019

Many CLIs use colons to separate subcommands instead of spaces, eg. heroku auth:login, heroku auth:logout, etc. It would be nice to have support for this sort of thing in clap as well. It becomes especially important when maintaining backwards compatibility with CLIs written in this format.

@CreepySkeleton CreepySkeleton added C: subcommands E-hard Call for participation: Experience needed to fix: Hard / a lot C-enhancement Category: Raise on the bar on expectations labels Feb 1, 2020
@pksunkara pksunkara added this to the 3.1 milestone Apr 9, 2020
@epage epage added A-builder Area: Builder API A-parsing Area: Parser's logic and needs it changed somehow. and removed C: subcommands labels Dec 8, 2021
@epage
Copy link
Member

epage commented Dec 8, 2021

So it sounds like you want to have heroku auth:login act like heroku auth logic today? Is there a reason to do this instead of just defining an auth:login subcommand? For help management?

@epage epage removed this from the 3.1 milestone Dec 8, 2021
@samuela
Copy link
Author

samuela commented Dec 9, 2021

Yeah, it's not a huge deal writing a CLI from scratch. Just a bit of an issue if you want to rewrite an existing CLI with clap.

@epage
Copy link
Member

epage commented Dec 9, 2021

I think what I'm missing is what the challenge is in porting an existing CLI to clap. In what way would a auth:login subcommand not work?

@samuela
Copy link
Author

samuela commented Dec 9, 2021

Honestly it's been a while since I was working on this, so I don't recall at this point. Things may have also changed with clap in the meantime that would obviate the need for this feature request.

@pksunkara
Copy link
Member

I think the intention is to treat auth as a command and then it having it's own subcommands (maybe for better organization in the help messages?).

@epage epage added S-waiting-on-design Status: Waiting on user-facing design to be resolved before implementing and removed P4: nice to have E-hard Call for participation: Experience needed to fix: Hard / a lot labels Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-builder Area: Builder API A-parsing Area: Parser's logic and needs it changed somehow. C-enhancement Category: Raise on the bar on expectations S-waiting-on-design Status: Waiting on user-facing design to be resolved before implementing
Projects
None yet
Development

No branches or pull requests

4 participants