sindresorhus/eslint-plugin-unicorn

Improve `prefer-dom-node-remove`

Geschlossen

#2.085 geöffnet am 09.05.2023

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (468 Forks)user submission
enhancementhelp wanted

Repository-Metriken

Stars
 (5.022 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 16h) (399 gemergte PRs in 30 T)

Beschreibung

  • Use snapshot test https://github.com/sindresorhus/eslint-plugin-unicorn/pull/2333
  • Improve fix, not use the original text but manipulate tokens
  • Improve fix for node?.parentNode.removeChild(node) (node can be anything, the object and the argument should be same reference, but can be different AST, eg: a?.b?.parentNode.remove(a.b))

Contributor Guide

Research-Richtung
Untersuche den vorhandenen Fixer in `rules/prefer dom node remove.js` (wahrscheinlich im Repository). Überprüfe den gemergten PR #2333 für den Snapshot Test Stil. Für die zweite Aufgabe ersetze die Verwendung von Originaltext durch Token Manipulation, indem du über `sourceCode.getTokens()` oder Ähnliches auf AST Token zugreifst. Für die dritte Aufgabe behandle optionales Chaining (z.B. `node?.parentNode.removeChild(node)`), stelle sicher, dass dieselbe Referenz sowohl im Objekt als auch im Argument verwendet wird; erwäge die Verwendung von `sourceCode.getText()` oder den Vergleich von AST Knoten. Sieh dir die vorhandenen Tests in `test/snapshots/prefer dom node remove.md` zur Orientierung an.
Tech Stack
javascriptnodejs
Domain
developer experiencetooling
Issue Type
Refactoring
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 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Experience with ESLint rulesUnderstanding of ASTFamiliarity with fixer API
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50