We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
There's a new class of quality issues in deck (see #331 and #288): cobra Commands wrongly overwriting or reading values meant for other commands.
Command
I propose that we improve maintainability/modularity of decK by refactoring cobra commands so that:
struct Config
I did something similar on one of my past projects: https://github.com/weaveworks/wksctl/blob/0184323ebc93df96544a1e3b55c84fbd3d292641/cmd/wksctl/main.go#L47
The text was updated successfully, but these errors were encountered:
Notes after a discussion with @hbagdi:
viper
Sorry, something went wrong.
No branches or pull requests
There's a new class of quality issues in deck (see #331 and #288): cobra
Command
s wrongly overwriting or reading values meant for other commands.I propose that we improve maintainability/modularity of decK by refactoring cobra commands so that:
struct Config
and has flags bound to a single instance of that structstruct Config
as its inputI did something similar on one of my past projects: https://github.com/weaveworks/wksctl/blob/0184323ebc93df96544a1e3b55c84fbd3d292641/cmd/wksctl/main.go#L47
The text was updated successfully, but these errors were encountered: