-
Notifications
You must be signed in to change notification settings - Fork 1
suggest to split client according to language #5
Comments
Hi, |
Hi @celrenheit sandglass-grpc have also suggestion, thanks your work! |
Watching for this to be done first before I build a dart client. |
@gedw99 I could help you with the dart client if needed, I built the node client and currently working on the python client |
I am currently building the python client, and get this error.
Can anyone help me here. The grpcio docs gives very little about this. |
That's really nice of you.
I have to put up the code onto GitHub. It's currently doing grpc between
golang and dart and was very easy to get working.
I still need to adapt it to sandglass do your domain knowledge in that area
is very helpful.
It would be smart to make the example code & tests match the node and
python examples so we can easily spot regressions.
Shall I let you know after I upload it today ?
…On Thu, 20 Sep 2018, 11:01 kaskerd, ***@***.***> wrote:
I am currently building the python client
<https://github.com/kaskerd/sandglass-client/tree/python>, and get this
error.
File "/Users/apple/sandglass-client/python_/sandglass_pb2_grpc.py", line 4, in <module>
import sandglass_pb2 as sandglass__pb2
File "/Users/apple/sandglass-client/python_/sandglass_pb2.py", line 20, in <module>
from github.aaakk.us.kg.gogo.protobuf.gogoproto import gogo_pb2 as github_dot_com_dot_gogo_dot_protobuf_dot_gogoproto_dot_gogo__pb2
Can anyone help me here. The grpcio docs gives very little about this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ATuCwkNcx8m1XI5r2X2n2_G1mCkaUefwks5uc1lbgaJpZM4WkhML>
.
|
@gedw99 Thanks, and sure let me know whenever you upload it, we might have time differences but that's fine. I like that you pointed out examples and "tests" especially, as I also need to test the node client, the only slight problem is sandglass isn't stable yet and might be redundant writing tests for now. The python client as you can see is nearly implemented. |
Hi, @celrenheit suggestion,
sandglass/sandglass-client
splits tosandglass/sandglass-go
,sandglass/sandglass-node
,sandglass/sandglass-java
,sandglass/sandglass-dart
, and so on.it have shorter
path/package-name
and convenient to auto test. thanks!The text was updated successfully, but these errors were encountered: