You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First of all, great library! It's great to see a 100% js alternative to the protobuf-for-node library.
I'm working with a .proto file that has top level enum definitions that are shared among two different messages. ProtoBuf.js claims "Illegal top level declaration", but Google's protoc compiles it just fine. I'm somewhat new to protobufs, but I would think if Google's protoc was ok with it, then it would be a good idea to follow suit.
The text was updated successfully, but these errors were encountered:
Thank you for reporting this! I was not sure if top-level enums are supported in protoc, so I might have skipped that. I will look into this and let you know.
First of all, great library! It's great to see a 100% js alternative to the protobuf-for-node library.
I'm working with a .proto file that has top level enum definitions that are shared among two different messages. ProtoBuf.js claims "Illegal top level declaration", but Google's protoc compiles it just fine. I'm somewhat new to protobufs, but I would think if Google's protoc was ok with it, then it would be a good idea to follow suit.
The text was updated successfully, but these errors were encountered: