sindresorhus/eslint-plugin-unicorn

Rule proposal: `prefer-map-has`

Chiusa

#1329 aperta il 3 giu 2021

Vedi su GitHubGuida contributor
 (8 commenti) (5 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
help wantednew ruletypes

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 4h 30m) (26 PR mergiate in 30 g)

Descrizione

When checking existence, should prefer Map#has() over Map#get().

Fail

if (map.get(foo));

Pass

if (map.has(foo));

I think better ship this with prefer-map together, so user can notice this problem when we fix object.foo to map.get(foo) in prefer-map.

Guida contributor

Direzione di ricerca
Studia le implementazioni delle regole ESLint esistenti nel plugin (ad esempio prefer map) per comprenderne la struttura, quindi crea una nuova regola che rilevi Map#get() nei contesti condizionali e suggerisca Map#has().
Tech stack
javascript
Dominio
tooling
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
JavaScriptESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80