fix: Better applyPatches type (#810)

joshkel · web-flow · commit 09ac09751371 · 2021-06-24T09:39:06.000+01:00
applyPatches should specify its input as type `T` so that TypeScript can better infer the correct `T` for its output. Fixes #809
diff --git a/src/core/immerClass.ts b/src/core/immerClass.ts
@@ -185,7 +185,7 @@ export class Immer implements ProducersFns {
 		this.useProxies_ = value
 	}
 
-	applyPatches<T extends Objectish>(base: Objectish, patches: Patch[]): T {
+	applyPatches<T extends Objectish>(base: T, patches: Patch[]): T {
 		// If a patch replaces the entire state, take that replacement as base
 		// before applying patches
 		let i: number

Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ export class Immer implements ProducersFns {`
`185`	`185`	`this.useProxies_ = value`
`186`	`186`	`}`
`187`	`187`
`188`		`- applyPatches<T extends Objectish>(base: Objectish, patches: Patch[]): T {`
	`188`	`+ applyPatches<T extends Objectish>(base: T, patches: Patch[]): T {`
`189`	`189`	`// If a patch replaces the entire state, take that replacement as base`
`190`	`190`	`// before applying patches`
`191`	`191`	`let i: number`