-
Notifications
You must be signed in to change notification settings - Fork 11
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 build options for filed type #69
Comments
@BeatsKitano This is intentional, as the default value fields (for example See https://developers.google.com/protocol-buffers/docs/proto3#json |
If the syntax = "proto3";
import "google/api/field_behavior.proto";
message Reply {
string id = 1 [
(google.api.field_behavior) = REQUIRED
];
} |
We've been discussing this internally, and landed on always setting type to For example |
|
Regarding 3, there is a comment including the field behavior, for example:
I'd be open to having build options for controlling this. Feel free to create a PR. |
This is a wonderful plugin! thank you for open source.
I have a little issue.this is my proto file hello.proto:
this lib generate filed id type to
string | undefined
, but field id is define.If id is optional, i think
string | undefined
is ok, now id is default required, the generate field type should bestring
, notstring | undefined
version: latest
The text was updated successfully, but these errors were encountered: