sindresorhus/p-memoize

Export "in-flight only memoizer"

Open

#53 ouverte le 1 févr. 2023

Voir sur GitHub
 (5 commentaires) (1 réaction) (1 assigné)TypeScript (441 stars) (32 forks)user submission
enhancementhelp wanted

Description

I don't know what this means:

pMemoize(fn, {cache: false})

I have to document it every time:

pMemoize(fn, {cache: false /* only while pending */})

The module can export a more readable helper instead:

export const pMemoizePending = (fn, opts) => pMemoize(fn, {...opts, cache: false})

Note: the types of opts should exclude cache

Some tags could also be added to package.json

Guide contributeur

Stack technique
typescript
Domaine
developer experience
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
The implementation requires adding a new exported function pMemoizePending in the main entry file (likely index.ts or src/index.ts). The function should accept fn and opts, excluding the cache option from opts. This is a wrapper around the existing pMemoize function. Review the existing TypeScript types for pMemoize to ensure the exclusion is correct. The package.json may also need tags added. Since the issue has an assignee, it is advisable to coordinate with them before starting work.