sindresorhus/eslint-plugin-unicorn

`unicorn/no-invalid-remove-event-listener` does not report when optional chaining

Chiusa

#2253 aperta il 12 gen 2024

Vedi su GitHubGuida contributor
 (1 commento) (3 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
bughelp wanted

Metriche repository

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

Descrizione

Similar to #2252 but on unicorn/no-invalid-remove-event-listener

Rule works correctly without optional chaining:

However with optional chaining stops reporting error:

this.input?.removeEventListener(event, this.onInputEvent.bind(this), true)

Can be easily tested by adding ? after el: https://github.com/sindresorhus/eslint-plugin-unicorn/blob/f0ff04d/test/no-invalid-remove-event-listener.mjs#L49

Guida contributor

Direzione di ricerca
Indaga l'implementazione della regola per gestire il concatenamento opzionale nell'oggetto chiamato. Modifica la regola per attraversare i nodi OptionalMemberExpression. Aggiungi un caso di test con concatenamento opzionale in test/no invalid remove event listener.mjs. Fai riferimento al problema simile #2252 come precedente.
Tech stack
javascriptnodejs
Dominio
toolingdeveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptESLint plugin internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65