search

mennanov / fieldmask-utils

Protobuf Field Mask Go utils
MIT License
229 stars 26 forks source link

Rewrite StructToMap (major update) #34

Closed mennanov closed 1 year ago

mennanov commented 1 year ago

This change updates the behavior of the StructToMap function in a way that it now supports more use cases (see https://github.com/mennanov/fieldmask-utils/issues/33 and https://github.com/mennanov/fieldmask-utils/pull/31).

This update introduces changes to the mapVisitor signature meaning that the existing code that uses it will have to be updated, otherwise it won't compile.

codecov[bot] commented 1 year ago

Codecov Report

Merging #34 (a0e2f97) into master (fc158cc) will decrease coverage by 3.30%. The diff coverage is 77.86%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
- Coverage   89.91%   86.61%   -3.30%     
==========================================
  Files           2        2              
  Lines         446      478      +32     
==========================================
+ Hits          401      414      +13     
- Misses         30       46      +16     
- Partials       15       18       +3
Impacted Files Coverage Δ
copy.go 83.18% <77.86%> (-4.36%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more