-
Notifications
You must be signed in to change notification settings - Fork 66
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
Support for optional fields #35
Comments
Is this issue still relevant? |
Hi! We are definitely still interested in this functionality. I feel like it should mainly be just updating proto version |
Which version of the plugin are you using? Seems that the plugin supports the |
if you want optional for a "null" field in proto3 all fields are optional in that sense, so it dont make sense, in proto itself, if its oneof its other case https://stackoverflow.com/questions/42622015/how-to-define-an-optional-field-in-protobuf-3 |
This should be fixed with #48 |
go generate ./...
api/assessment/metric.proto: is a proto3 file that contains optional fields, but code generator protoc-gen-gotag hasn't been updated to support optional fields in proto3. Please ask the owner of this code generator to support proto3 optional.--gotag_out:
proto.go:38: running "protoc": exit status 1
The text was updated successfully, but these errors were encountered: