-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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(module: upload): should get error raw response #2858
fix(module: upload): should get error raw response #2858
Conversation
Thanks for contribution @yosheng , please help us to confirm if it solve your issue. |
sure! @ElderJames |
Codecov ReportBase: 44.21% // Head: 44.25% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #2858 +/- ##
==========================================
+ Coverage 44.21% 44.25% +0.04%
==========================================
Files 545 545
Lines 25771 25771
Branches 261 261
==========================================
+ Hits 11394 11406 +12
+ Misses 14337 14325 -12
Partials 40 40
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@ElderJames how colud i fix this "Added line #L76 was not covered by tests" ? After looking through the project I don't know how to add unit tests |
You can ignore this warning. |
🤔 This is a ...
🔗 Related issue link
#2857
💡 Background and solution
📝 Changelog
☑️ Self Check before Merge