terraform-provider-iterative icon indicating copy to clipboard operation
terraform-provider-iterative copied to clipboard

Published 20 hours ago • verified publisher icon iterative

Some refactoring on leo subcommands

Open tasdomas opened this issue 1 year ago • 2 comments

Renamed delete to destroy. Renamed read to status. Restructured base options and moved all subcommands into a single package.

tasdomas avatar Sep 09 '22 16:09 tasdomas

moved all subcommands into a single package

Why? thinking

Is there any reason to keep identical single file packages?

tasdomas avatar Sep 09 '22 16:09 tasdomas

Is there any reason to keep identical single file packages?

~“Namespaces are one honking great idea — let's do more of those!”~ Wrong language! 🤭

It probably boils down to a matter of taste[^1] but, to my mind, it looks cleaner once we introduce subcommands or more complex commands (e.g. #291 or #531) which may be split over several files.

See also docker/cli, kubernetes/kubectl or cli/cli for inspiration.

[^1]: Or lack thereof, you're the wise gopher here.

0x2b3bfa0 avatar Sep 09 '22 16:09 0x2b3bfa0