bbs icon indicating copy to clipboard operation
bbs copied to clipboard

Use golang/protobuf instead of gogo/protobuf

Open ebroberson opened this issue 11 months ago • 1 comments

  • [X] Read the Contributing document.

Summary

gogo/protobuf has been deprecated for years. We should be using up-to-date protobufs provided by google.golang.org/protobuf. In order to fit our use cases, a protobuf plugin (protoc-gen-go-bbs) has been created to catch any customizations we want to add.

Backward Compatibility

Breaking Change? No

ebroberson avatar Feb 14 '25 20:02 ebroberson

I raised a couple questions, and fund a number of spots that could use either cleanup, or uncommenting depending on if the code is actually needed anymore.

I don't fully understand all the reasons for protoc-gen-bbs, and trying to understand how it's templating the generated code is breaking my brain.

This is very impressive, well done!

geofffranks avatar Feb 19 '25 18:02 geofffranks