-
Notifications
You must be signed in to change notification settings - Fork 2.2k
fix: broken link to extra deps documentation #2376
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2376 +/- ##
==========================================
+ Coverage 88.50% 88.82% +0.32%
==========================================
Files 230 230
Lines 12135 12135
==========================================
+ Hits 10740 10779 +39
+ Misses 1395 1356 -39
Flags with carried forward coverage won't be shown. Click here to find out more.
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👍
recheckcla |
All reactions
Sorry, something went wrong.
Jina CLA check ✅ All Contributors have signed the CLA. |
All reactions
Sorry, something went wrong.
I have read the CLA Document and I hereby sign the CLA |
All reactions
Sorry, something went wrong.
@akanz1 can you merge the latest master branch into your feature branch to pass the docstring check? |
All reactions
Sorry, something went wrong.
Latency summaryCurrent PR yields:
Breakdown
Backed by latency-tracking. Further commits will update this comment. |
All reactions
-
👀 1 reaction
Sorry, something went wrong.
@bwanglzu done, seems to pass now |
All reactions
-
👍 1 reaction
Sorry, something went wrong.
nan-wang
cristianmtr
florian-hoenicke
Successfully merging this pull request may close these issues.
No description provided.