TehShrike/deepmerge

d.ts type definition error

eczn opened this issue · 7 comments

eczn commented

image

⬆️ the type of t4.a should be a number type, not never.

What would be the correct way to fix the type definition? https://github.com/TehShrike/deepmerge/blob/master/index.d.ts#L1-L2

eczn commented

What would be the correct way to fix the type definition? https://github.com/TehShrike/deepmerge/blob/master/index.d.ts#L1-L2

#181 here

eczn commented

image

@TehShrike any plans to merge the PR?

Sorry for not looking in for a while – any other TS folks want to give #198 a review? @teppeis @RebeccaStevens @s-edlund

I can probably give it a review a bit later today

Sorry I took a bit longer to do this than I said I would but I just reviewed it. It's good but I found some issue with it; I've commented them there.