search

nilslice / protolock

Protocol Buffer companion tool. Track your .proto files and prevent changes to messages and services which impact API compatibility.
https://protolock.dev
BSD 3-Clause "New" or "Revised" License
604 stars 37 forks source link

Does protolock support proto2 ? #128

Closed doudoutinghua closed 4 years ago

doudoutinghua commented 4 years ago

when I remove a required field from the following file, protolock status is ok.

`syntax = "proto2";

message IdCard { reserved 1; reserved "id"; //required string id = 1; required string name = 2; optional string addr = 3; required string log = 4; } `

nilslice commented 4 years ago

The underlying proto parser supports Proto2, but protolock itself is not tested for any Proto2 support. I wouldn't count on it -- sorry!