Dario Castañé

Results 73 comments of 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?

@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.