-
Notifications
You must be signed in to change notification settings - Fork 12
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
Rnode 0.9 update: Either.proto etc. #62
Conversation
flow-check still in progress
all tests pass. yay!
@cboscolo it took me a while, but I think this is done now. |
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.
I've looked over all the changes, and seen travis tests pass. This looks ready to me. Great work.
@@ -273,23 +259,20 @@ function RNode(grpc /*: grpcT */, endPoint /*: { host: string, port: number } */ | |||
* JOINed set of names in the tuplespace | |||
* @memberof RNode | |||
* @param pars The names onwhich to listen | |||
* @return promise for ContinuationsWithBlockInfo | |||
* @return promise for DataWithBlockInfo |
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.
that looks suspect
good catch, Joshy
fixes #60