Dario Castañé
Dario Castañé
@ruanspies Not sure about this change.
@liggitt I regret I'm unable to fix this in a quick way. Can you provide a PR?
@mtt0 Why is it unexpected?
@xt99 The previous behavior caused a bug fixed by #125. Please use `mergo.WithOverwriteWithEmptyValue` to override with empty values in the next release 0.3.10.
@pckhoi @normanjaeckel Reopened, but I'm not able to dedicate time to fix it.
@tbnguyen1407 No, sorry. My current job is time-consuming and I'm not able to deal with issues without PRs.
@tbnguyen1407 Still the same situation. Would this code from @octobered help? https://github.com/octobered/mergo/commit/9e3abad214fff9790dd8b1fa2e7b0b1124d52de5
Reopening this issue as its PR introduced too many bugs.
@cwarden Sorry it took so long. The tests look fine, but I'm not sure if this PR can introduce unknown side effects. It looks you have been using your fork...
@eh-steve I'll check it. Thanks.