protobuf icon indicating copy to clipboard operation
protobuf copied to clipboard

Implemented optional (presented) fields for proto3

Open biosvs opened this issue 1 year ago • 2 comments

Ofc this PR will never be merged.

Although if you stuck with gogo, need "presence" feature, and brave enough..

(Added some fields to descriptor.proto manually. Didn't fix some tests - they are broken due to protoc version upgrade; didn't add new tests.)

fix #713

biosvs avatar Jul 27 '23 11:07 biosvs

Thanks

nightwolfz avatar Dec 30 '23 11:12 nightwolfz

Thanks!

~We consider using this until we get migrate out of gogoproto in Prometheus~ EDIT: Actually we decided to skip optional, it's too dangerous, even mentioned in the official Google Cloud docs 🙈

bwplotka avatar Feb 05 '24 10:02 bwplotka