-
Notifications
You must be signed in to change notification settings - Fork 71
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
Add ApplyEdge for SchemaManager #319
Conversation
wip WIP Update schema_manager_test.go fix fix
Will do some code refactoring after this PR has been merged :) |
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #319 +/- ##
==========================================
+ Coverage 64.32% 64.73% +0.41%
==========================================
Files 11 11
Lines 2643 2705 +62
==========================================
+ Hits 1700 1751 +51
- Misses 802 810 +8
- Partials 141 144 +3 ☔ View full report in Codecov by Sentry. |
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
* Add ApplyEdge wip WIP Update schema_manager_test.go fix fix * fix
* Add ApplyEdge wip WIP Update schema_manager_test.go fix fix * fix
See also #317
What type of PR is this?
What problem(s) does this PR solve?
Issue(s) number:
Description:
How do you solve it?
Special notes for your reviewer, ex. impact of this fix, design document, etc: