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

Group skywire-cli visor subcommands #1072

Closed
0pcom opened this issue Jan 27, 2022 · 0 comments
Closed

Group skywire-cli visor subcommands #1072

0pcom opened this issue Jan 27, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@0pcom
Copy link
Collaborator

0pcom commented Jan 27, 2022

We have a lot of subcommands in skywire-cli visor right now. Too many.

It would make sense to group some of these up into one visor subcommand with just additional subcommands or flags.

For example, skywire-cli visor apps should have either additional subcommands or flags for all the app-related subcommands in skywire-cli visor

the subcommands we need under skywire-cli visor are as follows

  • app
  • rule
  • tp
  • exec
  • pk
  • version
  • ip

so every visor subcommand (if there is more than one) related to one of the above listed items should be either a subcommand of the subcommand or a flag for the subcommand.

@0pcom 0pcom added the enhancement New feature or request label Jan 27, 2022
@jdknives jdknives changed the title group skywire-cli visor subcommands Group skywire-cli visor subcommands Jan 28, 2022
@jdknives jdknives closed this as completed Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants