bufbuild / protovalidate

Protocol Buffer Validation - Go, Java, Python, and C++ Beta Releases!
https://buf.build/bufbuild/protovalidate
Apache License 2.0
849 stars 37 forks source link

Migrator: properly handle ignore generation on fields #209

Closed rodaine closed 4 months ago

rodaine commented 4 months ago

ignore field migration (from ignore_empty and skipped) was preserving the field type (e.g., string or repeated) in the new format, which is erroneous. required was already handling this correctly, ignore just slipped through the cracks here.

Fixes #202

Thanks again @pquerna & @jschaf for the reports!