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

Deeply Nested Subcommand Scripts #28

Closed
Nekroze opened this issue Sep 27, 2018 · 0 comments
Closed

Deeply Nested Subcommand Scripts #28

Nekroze opened this issue Sep 27, 2018 · 0 comments
Assignees
Labels
Feature New feature or request for major functionality changes

Comments

@Nekroze
Copy link
Owner

Nekroze commented Sep 27, 2018

Is your feature request related to a problem? Please describe.
Currently subcommands are scripts in a single directory even though there are now many nested subcommands, this has made the scripts bloated and hard to grok/read/refactor.

Describe the solution you'd like
Allow directories under app/subcommands to become subcommand namespaces like dab repo is that does nothing but provide info and access to other subcommands represented by scripts in that directory.

Continuing with the repo example, app/subcommands/repo.sh would become something like app/subcommands/repo/{entrypoint,group,manage}.sh which would require the current repo add top level repo sub subcommands to be moved to something like manage.

Describe alternatives you've considered
Could rewrite in a different language that allows better composition, too big of an increment though.

Additional context
N/A.

@Nekroze Nekroze added the Feature New feature or request for major functionality changes label Sep 27, 2018
@Nekroze Nekroze self-assigned this Sep 27, 2018
Nekroze added a commit that referenced this issue Sep 27, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #28
Nekroze added a commit that referenced this issue Sep 27, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #28
Nekroze added a commit that referenced this issue Sep 27, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #28
Nekroze added a commit that referenced this issue Sep 27, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #28
Nekroze added a commit that referenced this issue Sep 27, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #28
Nekroze added a commit that referenced this issue Sep 27, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #28
Nekroze added a commit that referenced this issue Sep 29, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #24
Fixes issue #25
Fixes issue #28
Nekroze added a commit that referenced this issue Sep 29, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #24
Fixes issue #25
Fixes issue #28
Nekroze added a commit that referenced this issue Sep 29, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #24
Fixes issue #25
Fixes issue #28
Nekroze added a commit that referenced this issue Sep 29, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #24
Fixes issue #25
Fixes issue #28
Nekroze added a commit that referenced this issue Sep 29, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #24
Fixes issue #25
Fixes issue #28
Nekroze added a commit that referenced this issue Sep 30, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #24
Fixes issue #25
Fixes issue #28
Nekroze added a commit that referenced this issue Sep 30, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #24
Fixes issue #25
Fixes issue #28
Nekroze added a commit that referenced this issue Sep 30, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #24
Fixes issue #25
Fixes issue #28
Nekroze added a commit that referenced this issue Sep 30, 2018
A new way to define and manage subcommands, should also be more uniform.

Fixes issue #24
Fixes issue #25
Fixes issue #28
@Nekroze Nekroze closed this as completed Sep 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request for major functionality changes
Projects
None yet
Development

No branches or pull requests

1 participant