sindresorhus/eslint-plugin-unicorn

Rule proposal: Extend `prefer-modern-dom-apis` with `insertBefore` => `prepend`

Aperta

#1715 aperta il 4 feb 2022

Vedi su GitHubGuida contributor
 (2 commenti) (3 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
help wantednew rule

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 4h 30m) (26 PR mergiate in 30 g)

Descrizione

Description

Any usage of element.insertBefore(child, element.firstChild) can be converted to element.prepend(child).

(Note, this isn't safe with element.firstChildElement)

Fail

element.insertBefore(child, element.firstChild);

Pass

element.prepend(child)

Guida contributor

Direzione di ricerca
Studia la regola esistente `prefer modern dom apis` per comprenderne la struttura. Poi implementa una nuova regola che rilevi `insertBefore(child, element.firstChild)` e suggerisca `prepend(child)`. Usa i tipi di nodo AST di ESLint come CallExpression, MemberExpression, Identifier.
Tech stack
javascript
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ESLintJavaScriptAST
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75