-
Notifications
You must be signed in to change notification settings - Fork 33
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
Update example for cobra #70
Comments
[update] The following example works, for a new playground link.
|
Oh, great! Link: https://go.dev/play/p/zyXtrtL7NBu Do you want to make a PR ? 😄 (if so, I have started adding Thank you once more. |
I didn't make a PR because the thing I posted compiles and works, but outside the example it doesn't work super great having the configuration data and the command line arguments separated. |
Can you clarify what is exactly not ok? |
@cristaloleg I might be missing something, but with the cobra integration the configuration gets loaded into the aconfig "system", but the parameters get loaded into the cobra parameter system. For my use case I'd like just the aconfig system and to ignore the cobra stuff except for integration in the help text, etc. |
Current version is outdated https://play.golang.org/p/OsCR8qTCN0H
Probably there will be another repository which will contains all the examples with external dependencies for https://github.com/cristalhq libraries, with a CI on cron
The text was updated successfully, but these errors were encountered: