sindresorhus/eslint-plugin-unicorn

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

Fermée

#2 253 ouverte le 12 janv. 2024

Voir sur GitHubGuide contributeur
 (1 commentaire) (3 réactions) (0 personne assignée)JavaScript (468 forks)user submission
bughelp wanted

Métriques du dépôt

Stars
 (5 022 étoiles)
Métriques de merge PR
 (Merge moyen 1j 16h) (399 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Étudiez l'implémentation de la règle pour gérer le chaînage optionnel dans l'objet appelé. Modifiez la règle pour parcourir les nœuds OptionalMemberExpression. Ajoutez un cas de test avec chaînage optionnel dans test/no invalid remove event listener.mjs. Référez vous au problème similaire #2252 comme précédent.
Stack technique
javascriptnodejs
Domaine
toolingdeveloper experience
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptESLint plugin internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65