New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add syntax version to .proto files #220
Conversation
Thanks @shaneutt! Settings---
minimum_reviewers: 2
merge: true
build_steps:
- make clean
- make deps
- make compile
- make test
- make xref
- make dialyzer
org_mode: true
timeout: 1800 |
There seems to be an issue with build step **make_compile,make_test,make_xref,make_dialyzer** ! ☁️✅ MERGE
✅ MAKE_CLEAN
✅ MAKE_DEPS
⛔ MAKE_COMPILE
⛔ MAKE_TEST
⛔ MAKE_XREF
⛔ MAKE_DIALYZER
⬜ 0 of 2 Code reviews from organization basho |
The addition of |
@@ -13,6 +13,8 @@ | |||
* git add -A; git commit -m 'Update erlang code from .proto files' | |||
*/ | |||
|
|||
syntax = "proto2"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This syntax modifier is only valid in the Protocol Buffers 3 spec IIRC: https://developers.google.com/protocol-buffers/docs/proto3#simple
@@ -4145,6 +4145,7 @@ verify_msg(Msg, Opts) -> | |||
end. | |||
|
|||
|
|||
-dialyzer({nowarn_function,v_msg_mapfield/3}). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These will have to go in the dialyzer.ignore-warnings
file I think, probably a syntax that was added in R16 or R17.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a compilation issue. If you run protogen
using a "too new" Erlang, gpb
will insert these.
protoc
clients using libprotoc 3.0.0+ want.proto
files to specify their protocol buffers syntax version.This adds the syntax version to avoid warnings from libprotoc 3.0.0+.