-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[public-api] Switch go codegen to buf #13579
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
Conversation
c335b01
to
10909f9
Compare
/werft run 👍 started the job as gitpod-build-mp-public-api-buf.4 |
This appears to be blocking the merge pool for some reason even though it still has pending reviewers 😕 /hold |
7605389
to
49a714e
Compare
49a714e
to
6fa3d7c
Compare
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.
This also removes the vendored dependency on google/rpc as it's not needed.
I don't really understand what is happening in the PR. I thought we are using google rpc because of error handling? Is it not needed anymore or it is still generated differently?
The default libraries (TS, go, etc..) which use the code already embed the google RPC structs in their responses and in the protocol. We don't need to vendor them here (we're also not importing them into our protos). |
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.
lgtm
/unhold |
Description
Uses buf gen to generate go/grpc
This makes the configuration easier to read and removes the need to move files around.
This also removes the vendored dependency on google/rpc as it's not needed.
We can also the same for other packages.
Related Issue(s)
Fixes #
How to test
Release Notes
Documentation
Werft options:
If enabled this will build
install/preview
Valid options are
all
,workspace
,webapp
,ide