Fix Keyv.Options type incorrectly defines store option type

jaredwray · Jul 12, 2022 · b381679 · b381679
1 parent 6df4fb3
commit b381679
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/keyv/src/index.d.ts b/packages/keyv/src/index.d.ts
@@ -79,7 +79,7 @@ declare namespace Keyv {
 		/** The connection string URI. */
 		uri?: string | undefined;
 		/** The storage adapter instance to be used by Keyv. */
-		store?: Store<Value> | undefined;
+		store?: Store<string | undefined> | undefined;
 		/** Default TTL. Can be overridden by specififying a TTL on `.set()`. */
 		ttl?: number | undefined;
 		/** Specify an adapter to use. e.g `'redis'` or `'mongodb'`. */

diff --git a/packages/redis/src/index.d.ts b/packages/redis/src/index.d.ts
@@ -10,7 +10,7 @@ declare class KeyvRedis extends EventEmitter implements Store<string | undefined
 	constructor(options?: KeyvRedis.Options | Redis | Cluster);
 	constructor(uri: string | Redis | Cluster, options?: KeyvRedis.Options);
 	get(key: string): Promise<string | undefined>;
-	getMany(keys: string[]): Promise<string[] | undefined>;
+	getMany(keys: string[]): Promise<string[]>;
 	set(key: string, value: string | undefined, ttl?: number): Promise<any>;
 	delete(key: string): boolean;
 	deleteMany(keys: string[]): boolean;