-
Notifications
You must be signed in to change notification settings - Fork 721
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 missing calls to .Error() (#2870) #2874
Conversation
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@howardlau1999 please accept the invitation then you can push to the cherry-pick pull requests. |
/merge |
/run-all-tests |
Codecov Report
@@ Coverage Diff @@
## release-4.0 #2874 +/- ##
===============================================
- Coverage 77.42% 77.39% -0.04%
===============================================
Files 208 208
Lines 22762 22762
===============================================
- Hits 17624 17617 -7
- Misses 3812 3817 +5
- Partials 1326 1328 +2
Continue to review full report at Codecov.
|
/run-all-tests |
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@ti-srebot merge failed. |
/merge |
/run-all-tests |
/merge |
2 similar comments
/merge |
/merge |
@ti-srebot merge failed. |
/run-all-tests |
@ti-srebot merge failed. |
/run-all-tests |
@ti-srebot merge failed. |
/run-all-tests |
@ti-srebot merge failed. |
/merge |
/run-all-tests |
@ti-srebot merge failed. |
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
/run-all-tests |
/run-all-tests |
/run-all-tests |
cherry-pick #2870 to release-4.0
Signed-off-by: Howard Lau howardlau1999@hotmail.com
What problem does this PR solve?
We should call
.Error
to converterror
tostring
, otherwise we will get an empty result.What is changed and how it works?
Add missing
.Error()
call for API error responses.Check List
Tests
Release note