-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Refactor] Move command runner to common #63
Comments
actually the runner is highly coupled with viper, so move it non trivial, but the |
also may need sth like a context, in order to extend for things like run using shell instead of |
About the register command dynamically on cobra,
Adding things in However, since cobra is executed only when |
the fallback is, for Ayi, always read these default config files and generate dynamic commands ... actually it's quite overkill |
Ansible provide a good example, http://docs.ansible.com/ansible/playbooks_intro.html
Actually Ayi's intention is quite like Ansible |
currently, only part of the runner code is in
util/runner
, also command.go is coupled withviper
, and I plan to split it out like the log package #59, so it can be used in other projects.common/runner
,app/runner
maybe betterls
The text was updated successfully, but these errors were encountered: