sindresorhus/eslint-plugin-unicorn

Rule proposal: `prefer-map-has`

Geschlossen

#1.329 geöffnet am 03.06.2021

Auf GitHub ansehenContributor Guide
 (8 Kommentare) (5 Reaktionen) (0 zugewiesene Personen)JavaScript (468 Forks)user submission
help wantednew ruletypes

Repository-Metriken

Stars
 (5.022 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 4h 30m) (26 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Studieren Sie vorhandene ESLint Regelimplementierungen im Plugin (z. B. prefer map), um die Struktur zu verstehen, und erstellen Sie dann eine neue Regel, die Map#get() in Bedingungskontexten erkennt und Map#has() vorschlägt.
Tech Stack
javascript
Domain
tooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptESLint
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80