sindresorhus/eslint-plugin-unicorn

Configurable callee blacklist for `no-fn-reference-in-iterator`

Chiusa

#787 aperta il 29 giu 2020

Vedi su GitHubGuida contributor
 (2 commenti) (1 reazione) (0 assegnatari)JavaScript (468 fork)user submission
enhancementhelp wanted

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 1g 16h) (399 PR mergiate in 30 g)

Descrizione

I noticed that the rule currently has a hard-coded blacklist of callees to ignore.

I ran into a bit of an issue with this as we're currently moving our code base off of bluebird (callee: Promise) to async / await + blend-promise-utils (callee: P). That switch from Promise.map to P.map started throwing false-positives all over our codebase.

What are people's thoughts on making this a configurable option? I'd be willing to take a stab at a PR for it if so.

Guida contributor

Direzione di ricerca
Esamina il file di regola `rules/no fn reference in iterator.js`, in particolare l'array di blacklist hardcoded. Implementa un'opzione di configurazione (ad esempio `allowedCallees`) per consentire agli utenti di sovrascrivere o estendere la blacklist. Aggiorna il file di test corrispondente e la documentazione. Il maintainer non ha fornito requisiti API specifici, quindi proponi un design nell'issue prima dell'implementazione.
Tech stack
javascript
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptESLint plugin development
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60