-
Notifications
You must be signed in to change notification settings - Fork 17
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
Port to v2 #389
Port to v2 #389
Conversation
Signed-off-by: Sri Krishna <[email protected]>
Signed-off-by: Sri Krishna <[email protected]>
Signed-off-by: Sri Krishna <[email protected]>
Hey @srikrsna-buf , any update on this? Would love to see it merged! 😄 Thanks |
@itsjoeoui, we want to release connect-es, connect-query-es, and connect-playwright-es with support for v2, and update the documentation where necessary. We plan to release it at the same time to make it easier to upgrade. That's why this PR isn't active right now, but we are actively working on the release 🙂 |
Awesome! Thanks for the update 👍 |
return create( | ||
methodSig.output, | ||
( | ||
updater as (prev?: MessageShape<O>) => MessageInitShape<O> | undefined |
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.
Weird cast, is this even needed?
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.
Yes.
Unsafe argument of type
any
assigned to a parameter of typeMessageInitShape<DescMessage & O> | undefined
@typescript-eslint/no-unsafe-argument
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.
Ah weird. My IDE didn't have any problems. Good to know.
Signed-off-by: Sri Krishna <[email protected]>
Signed-off-by: Sri Krishna <[email protected]>
Port to v2 of connect and protobuf. User code doesn't change unless using some internal types and functions.