MergeDeep: fixed optional when value type is any or never

[Emiyaaaaa]

• Fixed MergeDeep removes optionals from any and never fields #775

I think @skarab42 uses EnforceOptional to restore the '?' by check undefined, because he dosen't used keyof in the implementation.

Now I used keyof to resolve the issue #775 , thus EnforceOptional is useless, so I seem need remove it, but it will cause some case changed like:

MergeDeep<{a: string | undefined}, {}>
// Before: => {a?: string | undefined}
// After:  => {a: string | undefined}

But I still think I should remove EnforceOptional, what do you think? @sindresorhus

[sindresorhus]

thus EnforceOptional is useless, so I seem need remove it, but it will cause some case changed like:

That is a bug fix. The new behavior is the correct one.

But I still think I should remove EnforceOptional, what do you think?

👍

[Emiyaaaaa]

thus EnforceOptional is useless, so I seem need remove it, but it will cause some case changed like:

That is a bug fix. The new behavior is the correct one.

But I still think I should remove EnforceOptional, what do you think?

👍

got it, it's ready for review