search

yeqown / protoc-gen-fieldmask

Generate FieldMask utility functions for protobuf
MIT License
34 stars 9 forks source link

Use actual field name instead FieldMask prefix for field mask method #10

Closed vaidasn closed 2 years ago

vaidasn commented 2 years ago

Use actual field name prefix instead fixed FieldMaskWithMode, FieldMask_Filter, and FieldMask_Prune names. Put underscore for WithMode method name to make it consistent with other generated methods. Fix generated _Filter method documentation.

Fixes #9

yeqown commented 2 years ago

I don't think so, use fixed prefix FieldMask_ is clear for user to distinguish between other plugin generated functions

yeqown commented 2 years ago

It also should have no relevance to the FieldMask field.