-
Notifications
You must be signed in to change notification settings - Fork 800
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
Go client patch #636
Go client patch #636
Conversation
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.
😺 Thanks for your contribution. There are some suggestions on this PR.
Codecov ReportBase: 70.50% // Head: 70.50% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #636 +/- ##
=======================================
Coverage 70.50% 70.50%
=======================================
Files 63 63
Lines 12495 12495
=======================================
Hits 8810 8810
Misses 2837 2837
Partials 848 848 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. |
@zhenghaoz I had done the requested changes, can you review again? |
Thanks for your contribution. This PR will be merged once all tests pass. |
Added Patch to Go Client for User and Item