-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
feat(hub): add --verbose
option to "jina hub push" CLI
#4278
Conversation
Latency summaryCurrent PR yields:
Breakdown
Backed by latency-tracking. Further commits will update this comment. |
95787e4
to
dbf4519
Compare
Codecov Report
@@ Coverage Diff @@
## master #4278 +/- ##
==========================================
- Coverage 87.73% 87.66% -0.08%
==========================================
Files 113 113
Lines 8254 8311 +57
==========================================
+ Hits 7242 7286 +44
- Misses 1012 1025 +13
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
dbf4519
to
34e7c14
Compare
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.
Require a unit test to cover this case.
34e7c14
to
bbd0c49
Compare
Added checks on unit tests |
Close: https://github.com/jina-ai/hubble/issues/203