stack: add --namespace option to the install command

crossplane / crossplane-cli

Tools and commands for managing and developing Crossplane

Apache License 2.0

14 stars 12 forks source link

stack: add --namespace option to the install command #27

Closed displague closed 4 years ago

displague commented 4 years ago

Argument parsing for the crossplane stack install command can now take arguments in any order. Positional parameters have their order preserved. https://stackoverflow.com/a/14203146/156674 was used as a reference.

All of the following are supported:

--namespace=foo
-n=foo
--namespace foo
-n foo

Closes #26

displague commented 4 years ago

What testing has been done? : )

I successfully installed a namespaced and cluster scoped stack using the short arguments, long arguments, equal arguments, and switched the parameters around.

suskin commented 4 years ago

What testing has been done? : )

I successfully installed a namespaced and cluster scoped stack using the short arguments, long arguments, equal arguments, and switched the parameters around.

Did you already try it with no argument?

displague commented 4 years ago

Running install with no arguments fell through to the generate-install help page.

I've pushed 2500ad3 which addresses this with a "Missing arguments" warning and usage notes.