-
Notifications
You must be signed in to change notification settings - Fork 496
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
[TUBEMQ-447] Add Broker-Admin Cli #347
Conversation
gosonzhang
commented
Dec 11, 2020
- Add Broker-Admin's Cli command, directly enter the command(deficiency: I want to implement the command and then bring the sub-command mode, and found that commons-cli cannot support my own ideas, temporarily adopt this plan, and adjust it when there is a better plan );
- Modify some implementation bugs of web-api
Codecov Report
@@ Coverage Diff @@
## master #347 +/- ##
===========================================
- Coverage 7.80% 7.77% -0.04%
Complexity 466 466
===========================================
Files 258 260 +2
Lines 27892 28000 +108
Branches 4579 4597 +18
===========================================
Hits 2177 2177
- Misses 25242 25350 +108
Partials 473 473
Continue to review full report at Codecov.
|
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.
LGTM
Co-authored-by: gosonzhang <gosonzhang@tencent.com>
Co-authored-by: gosonzhang <gosonzhang@tencent.com>
Co-authored-by: gosonzhang <gosonzhang@tencent.com>
…#347) Co-authored-by: v_haifxu <v_haifxu@tencent.com>