sindresorhus/eslint-plugin-unicorn

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

Open

#2.491 geöffnet am 28. Okt. 2024

Auf GitHub ansehen
 (1 Kommentar) (2 Reaktionen) (0 zugewiesene Personen)JavaScript (5.022 Stars) (468 Forks)user submission
enhancementhelp wanted

Beschreibung

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);

Contributor Guide

Tech Stack
javascript
Domain
toolingdeveloper experience
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaScriptESLint rule basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75
Research-Richtung
The issue requests adding a `minimumItems` option to the `prefer set has` rule. The rule is located in `rules/prefer set has.js`. The option requires modifying the rule's `schema` property and updating the logic to check the array length against `minimumItems`. Tests should be added in the relevant test directory, and documentation in `docs/rules/prefer set has.md` should be updated. The single comment on the issue may provide additional context.