Skip to content

Conversation

@smacker
Copy link
Contributor

@smacker smacker commented Dec 13, 2018

Locked version is causing error when an application needs to use newer
compatible grpcio.

Ref: bblfsh/python-client#138

Signed-off-by: Maxim Sukharev [email protected]

@smacker
Copy link
Contributor Author

smacker commented Dec 13, 2018

/cc @vmarkovtsev you may want to update bblfsh & grpcio in ML projects too. Especially in style-analyzer. I found this bug in sonar-analyzer: grpc/grpc#14088 which is fixed in newer grpcio.

@carlosms carlosms requested a review from vmarkovtsev December 13, 2018 11:33
@dpordomingo dpordomingo removed their request for review December 13, 2018 12:28
@dpordomingo
Copy link
Contributor

dpordomingo commented Dec 17, 2018

@smacker if you rebase onto the current master you will be benefited of #50 so one of the Travis stage will pass.

Locked version is causing error when an application needs to use newer
compatible grpcio.

Ref: bblfsh/python-client#138

Signed-off-by: Maxim Sukharev <[email protected]>
@smacker
Copy link
Contributor Author

smacker commented Dec 17, 2018

bblfsh/python-client#139 😅

@smacker smacker merged commit 0cf0d54 into src-d:master Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants