sindresorhus/eslint-plugin-unicorn

Improve `prefer-dom-node-remove`

Chiusa

#2085 aperta il 9 mag 2023

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
enhancementhelp wanted

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 1g 16h) (399 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina il correttore esistente in `rules/prefer dom node remove.js` (probabilmente nel repository). Rivedi la PR #2333 unita per lo stile dei test snapshot. Per il secondo compito, sostituisci l'uso del testo originale con la manipolazione dei token accedendo ai token AST tramite `sourceCode.getTokens()` o simile. Per il terzo compito, gestisci l'optional chaining (es. `node?.parentNode.removeChild(node)`) assicurandoti che lo stesso riferimento sia utilizzato sia nell'oggetto che nell'argomento; considera l'uso di `sourceCode.getText()` o il confronto dei nodi AST. Guarda i test esistenti in `test/snapshots/prefer dom node remove.md` per una guida.
Tech stack
javascriptnodejs
Dominio
developer experiencetooling
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Experience with ESLint rulesUnderstanding of ASTFamiliarity with fixer API
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50