-
Notifications
You must be signed in to change notification settings - Fork 432
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
Add a Cobra root command. #1908
Conversation
bcde80e
to
f704a23
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1908 +/- ##
==========================================
- Coverage 61.30% 61.22% -0.09%
==========================================
Files 129 130 +1
Lines 12339 12350 +11
==========================================
- Hits 7565 7561 -4
- Misses 4434 4448 +14
- Partials 340 341 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
* This is to ease the migration from urfave/cli to Cobra + Viper for configuration. * The new config will only be triggered if ENABLE_VIPER_CONFIG env var is set to true.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
linux tests are failing, please check before merge.
Description
Link to the issue in case of a bug fix.
NA
Testing details