sindresorhus/memoize

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

Open

#90 aperta il 1 feb 2023

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)TypeScript (55 fork)user submission
enhancementhelp wanted

Metriche repository

Star
 (1155 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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
})

Guida contributor

Direzione di ricerca
Aggiungere un console.warn nella funzione memoize quando la funzione memorizzata accetta più di un argomento e non viene fornita l'opzione cacheKey. L'avviso dovrebbe suggerire di utilizzare un cacheKey personalizzato.
Tech stack
typescriptjavascript
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jsnpm
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80