Skip to content
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

@grpc/grpc-js compatibility #192

Closed
ackdom opened this issue Jul 14, 2020 · 5 comments · Fixed by #220
Closed

@grpc/grpc-js compatibility #192

ackdom opened this issue Jul 14, 2020 · 5 comments · Fixed by #220

Comments

@ackdom
Copy link

ackdom commented Jul 14, 2020

Hi I have a question, we would like to migrate to grps-js package and I would like to know if and when you are planning support for it since it is not compatible right now

https://www.npmjs.com/package/@grpc/grpc-js#migrating-from-grpc

I guess only because of the start / startAsync which should not be such a problem.

I am interested in your point of view

@mxzinke
Copy link

mxzinke commented Aug 10, 2020

The original grpc package is running into deprecated in less than one year. Maybe it makes sense to change the dependencies to use the @grpc/grpc-js package. After evaluating how much has to be changed, I could do a PR.

But the migration from grpc, @dominoo send in looks great to bring in.

Any words on why not to do it?

@civilizeddev
Copy link

Is there anyone who can lead this?

@bojand
Copy link
Member

bojand commented Aug 11, 2020

Thanks for creating this issue. Will try and get to this soon.

@yoitsro
Copy link

yoitsro commented Oct 6, 2020

Hey @bojand! Let us know if there's anything we can do to help this along :)

@HappyZombies
Copy link

Any update on this? April is near! I might just make a PR...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants