-
Notifications
You must be signed in to change notification settings - Fork 67
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
Refactor Client Code, part 3/7: core-data #198
Conversation
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
Codecov Report
@@ Coverage Diff @@
## master #198 +/- ##
==========================================
- Coverage 48.39% 45.96% -2.44%
==========================================
Files 64 64
Lines 2093 2145 +52
==========================================
- Hits 1013 986 -27
- Misses 1002 1074 +72
- Partials 78 85 +7
Continue to review full report at Codecov.
|
Signed-off-by: Brandon Forster <me@brandonforster.com>
recheck |
Signed-off-by: Brandon Forster <me@brandonforster.com>
@michaelestrin let me know if this looks good and if so, I'll kick off the rebase cascade and update the names of the URL variables to urlPrefix. |
Signed-off-by: Brandon Forster <me@brandonforster.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.
Code changes look fine. I'll defer to others that know this code better on final approval.
PR content included in #202. |
This PR is the third part of several working on #196.
This PR refactors all of core-data's clients and updates their unit tests.
To test, actuate the endpoints in the core-data REST client.