-
Notifications
You must be signed in to change notification settings - Fork 99
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
Use errors.New instead of fmt.Errorf #127
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #127 +/- ##
===================================================
+ Coverage 85.54052% 85.55294% +0.01242%
===================================================
Files 188 188
Lines 16114 16114
===================================================
+ Hits 13784 13786 +2
+ Misses 1771 1763 -8
- Partials 559 565 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
CLA check doesn't seem to be working properly. Could you post a new comment? @chenyahui |
Co-authored-by: wineandchord <guoqizhou123123@qq.com>
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
I have read the CLA Document and I hereby sign the CLA |
I have read the CLA Document and I hereby sign the CLA