Rule proposal: Use `Array#includes()` instead of repeated conditional checks

Metriche repository

Star

5022 stelle (5022 stelle)

Metriche merge PR

Merge medio 1g 16h (Merge medio 1g 16h)399 PR mergiate in 30 g (399 PR mergiate in 30 g)

Descrizione

const [, , ...args] = process.argv;
if (args[0] === '-h' || args[0] === '--help') {
  outputHelp();
} 

const [, , ...args] = process.argv;
if (['-h', '--help'].includes(args[0])) {
  outputHelp();
}

Guida contributor

Direzione di ricerca

Implementa la regola 'use array includes' che controlla i controlli condizionali OR multipli con la stessa variabile e suggerisce l'uso di Array.includes(). Fai riferimento a regole esistenti come 'prefer includes' o simili nel repository eslint plugin unicorn. Esamina i file di implementazione delle regole nella directory `rules/`. La regola dovrebbe utilizzare un selettore AST per `IfStatement` e controllare il test per pattern come `a === b || a === c`. Fornisci un fixer per sostituire con `[b, c].includes(a)`.

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.

Mezza giornata

Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.

Datata

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Abbastanza chiara

Prerequisiti

Basic knowledge of ESLint rulesFamiliarity with AST selectorsUnderstanding of array methods

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

50