Open ikstewa opened 6 months ago
After https://github.com/chrusty/protoc-gen-jsonschema/commit/11e14e6de6c0d7f7f6d45b173735546b77f548ea the all_fields_required flag respects the proto3 optional keyword.
all_fields_required
optional
According to the proto3 JSON Mapping a null json array is treated the same as an empty json array. I think it would make sense to follow the pattern with the proto3 optionl and also not mark repeated fields as required when mapping to json schema.
repeated
After https://github.com/chrusty/protoc-gen-jsonschema/commit/11e14e6de6c0d7f7f6d45b173735546b77f548ea the
all_fields_requiredflag respects the proto3optionalkeyword.According to the proto3 JSON Mapping a null json array is treated the same as an empty json array. I think it would make sense to follow the pattern with the proto3 optionl and also not mark
repeatedfields as required when mapping to json schema.