-
Notifications
You must be signed in to change notification settings - Fork 207
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
Implements full spec of gRPC endpoint. #1042
Conversation
12e5047
to
c0af545
Compare
Signed-off-by: Artur Souza <asouza.pro@gmail.com>
c0af545
to
c061074
Compare
Signed-off-by: Artur Souza <asouza.pro@gmail.com>
Signed-off-by: Artur Souza <asouza.pro@gmail.com>
Signed-off-by: Artur Souza <asouza.pro@gmail.com>
89d782f
to
45be0ef
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1042 +/- ##
============================================
+ Coverage 76.91% 77.79% +0.88%
+ Complexity 1592 1570 -22
============================================
Files 145 144 -1
Lines 4843 4797 -46
Branches 562 563 +1
============================================
+ Hits 3725 3732 +7
+ Misses 821 780 -41
+ Partials 297 285 -12 ☔ View full report in Codecov by Sentry. |
ports: | ||
- "2181:2181" | ||
- 22181:2181 |
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.
is the extra 2 there intentional? in 22181
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
Description
Implements full spec of gRPC endpoint.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #1028
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: