You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.
After reviewing #23, I'm thinking the API provided by this package is not ideal since it requires passing things down into the cobra command that can't even be configured through the CLI anyway.
Instead of this package encapsulating the controllers, we should probably just have it surface the configurations for the controllers.
Or at least have a multi-stage approach that doesn't involve passing unrelated configuration into the CLI code.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
After reviewing #23, I'm thinking the API provided by this package is not ideal since it requires passing things down into the cobra command that can't even be configured through the CLI anyway.
Instead of this package encapsulating the controllers, we should probably just have it surface the configurations for the controllers.
Or at least have a multi-stage approach that doesn't involve passing unrelated configuration into the CLI code.
The text was updated successfully, but these errors were encountered: