jhump/protoreflect

Use proto v2 for rendering options in output

marcoferrer opened this issue · 1 comments

The current proto text printer outputs options using <> to denote the open and close of a message literal. The newer proto printer in the V2 API uses {} which is the most common convention used in public examples.

https://github.com/grpc-ecosystem/grpc-gateway/blob/master/examples/internal/proto/examplepb/a_bit_of_everything.proto

PR with proposed changes
#530

jhump commented

The formatting is much improved in #354. This does not actually use the proto v2 package for rendering but instead uses a custom renderer for message literals that can respect the line breaking configuration of the printer.