sindresorhus/eslint-plugin-unicorn

`prefer-set-has`: Add a `minimumItems` array length option

Open

Aperta il 28 ott 2024

Vedi su GitHub
 (1 commento) (2 reazioni) (0 assegnatari)JavaScript (5022 star) (468 fork)user submission
enhancementhelp wanted

Descrizione

Description

Add a minimumItems option so only arrays from a certain known size will be linted (default 0).

The rule can sometimes be too strict, using sets over arrays is usually more performant the more elements it has, and it's negligible in small numbers and readability is preferred.

Fail

minimumItems: 5

const array = [1, 2, 3, 4, 5, 6, 7];
const hasValue = value => array.includes(value);

Pass

minimumItems: 5

const array = [1, 2];
const hasValue = value => array.includes(value);

Guida contributor

Tech stack
javascript
Dominio
toolingdeveloper experience
Tipo issue
feature
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
JavaScriptESLint rule basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75
Direzione di ricerca
La richiesta propone di aggiungere un'opzione `minimumItems` alla regola `prefer set has`. La regola si trova in `rules/prefer set has.js`. L'opzione richiede la modifica della proprietà `schema` della regola e l'aggiornamento della logica per controllare la lunghezza dell'array rispetto a `minimumItems`. Dovrebbero essere aggiunti test nella directory di test pertinente e la documentazione in `docs/rules/prefer set has.md` dovrebbe essere aggiornata. Il singolo commento sulla richiesta potrebbe fornire ulteriore contesto.