-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Closed
Labels
Description
What is the problem you're trying to solve
I've been upgrading protobuf in Protobuild and ttrpc to use Google's latest protobuf package instead of gogo/protobuf.
However protoc-gen-gogoctrd seems a blocker for the migration. It is replacing Id -> ID to better integrate with Go's idiomatic naming convention. There are few requests regarding the feature.
- Go lint naming compliance golang/protobuf#53
- protoc-gen-go: support go_name option to specify Go identifier golang/protobuf#555
But both of them have been rejected.
Describe the solution you'd like
We need to either
- Use https://github.com/alta/protopatch to have Id -> ID conversion.
- Fork protoc-gen-go to support Id -> ID conversion.
- Give up the Id -> ID conversion.
Additional context
No response