Closed joshuaghezzi closed 6 months ago
I don't believe that trailing semicolons on blocks are valid Protobuf syntax.
From GPT4:
From the Buf proto language spec:
Block declarations: A block declaration is used for composite elements: those that may contain other elements. Instead of ending with a semicolon, they end with a block enclosed in braces ({ and }). The contained elements are defined inside these braces.
This appears to apply for both proto2 and proto3. As such, I'm inclined to reject this PR.
I would disagree with the GPT4 output, I believe they are valid optional syntax. At least in terms of being able to compile that is.
Unfortunately if this is not supported this package would be unusable for my use case as our proto method block declarations do end with a semicolon.
What are you using to codegen from the protos? It's possible that Buf's spec is incorrect--it's not canonical. protoc is canonical.
Yeah seems like it could be the case. Just using protoc
to generate source code.
Ok, let's merge it. Unfortunately, there's a merge conflict. Can you rebase for me?
Merged and released in 1.3.2!
Thanks ❤️