Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@johansja
Copy link

@johansja johansja commented Oct 15, 2025

Description

Adding protoc-gen-connect-go to generate ConnectRPC server and client stub.

Issue reference

Help with adding a ConnectRPC based service implementation in go-sdk discussed in dapr/go-sdk#786

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

@johansja johansja requested review from a team as code owners October 15, 2025 01:24
@JoshVanL
Copy link
Contributor

Thanks @johansja, please can you do DCO signoff?

@johansja
Copy link
Author

I hope I did it correctly by following instructions from here: https://docs.dapr.io/contributing/contributing-overview/#developer-certificate-of-origin-signing-your-work

@johansja
Copy link
Author

Hi @JoshVanL @mikeee is there anything else that I need to do to get this merged?

@JoshVanL
Copy link
Contributor

I would like to hear from @cicoyle and @yaron2 on what they think about including this- from my side, this seems like a useful non-functional addition which doesn't increase maintenance by much. (?)

@johansja
Copy link
Author

@cicoyle @yaron2 Any thought on this?

@yaron2
Copy link
Member

yaron2 commented Oct 24, 2025

I would like to hear from @cicoyle and @yaron2 on what they think about including this- from my side, this seems like a useful non-functional addition which doesn't increase maintenance by much. (?)

Yes this seems fine, especially as ConnectRPC is a CNCF project

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.

4 participants