sindresorhus/memoize

Warn when `cacheKey` is unset and `function.length > 1`?

Open

#90 opened on 2023年2月1日

GitHub で見る
 (6 comments) (0 reactions) (0 assignees)TypeScript (55 forks)user submission
enhancementhelp wanted

Repository metrics

Stars
 (1,155 stars)
PR merge metrics
 (30d に merged PR はありません)

説明

By default, only the memoized function's first argument is considered via strict equality comparison

I think mem should call console.warn in this case:

mem(async (host, path) => fetch(host + path))

Correct usage:

mem(async (host, path) => fetch(host + path), {
  cacheKey: args => args.join()
})
mem(async (host, path) => fetch(host + path), {
  cacheKey: JSON.stringify
})

コントリビューターガイド