-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
fix: fix hasCollection response has no status #37254
Conversation
@wayblink Please associate the related issue to the body of your Pull Request. (eg. “issue: #”) |
@wayblink E2e jenkins job failed, comment |
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
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #37254 +/- ##
==========================================
- Coverage 83.15% 80.88% -2.27%
==========================================
Files 1029 1319 +290
Lines 157251 182666 +25415
==========================================
+ Hits 130764 147754 +16990
- Misses 21322 29734 +8412
- Partials 5165 5178 +13
|
/run-cpu-e2e |
@wayblink E2e jenkins job failed, comment |
7181b04
to
5284622
Compare
Signed-off-by: wayblink <[email protected]>
5284622
to
56b4b77
Compare
@wayblink cpp-unit-test check failed, comment |
rerun cpp-unit-test |
@wayblink cpp-unit-test check failed, comment |
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
rerun cpp-unit-test |
@wayblink cpp-unit-test check failed, comment |
1 similar comment
@wayblink cpp-unit-test check failed, comment |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: congqixia, czs007, wayblink The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
issue: #37257