-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
API module rework #18847
Comments
before the final step we should get an adr which defines what are the rules around protobuf we are adhering to. We dont follow protobuf rules and the mix of following and not following is causing issues. We should squash this once and for all |
Maybe we don't need an api module at all, and solely use buf autogenerated code: https://buf.build/docs/bsr/generated-sdks/go with the https://buf.build/protocolbuffers/go plugin. |
Closing this until we have an actual proposal. We can get away long enough by using buf.build generated sdk and setup buf plugin and improve our buf usage altogether (#19503) |
Having a global API module for separately versioned modules isn't correct.
<To elaborate>
Work breakdown
ref: #12003
Once this issue is closed, we will be able to backport #18473
The text was updated successfully, but these errors were encountered: