sindresorhus/eslint-plugin-unicorn

Rule proposal: mistaken parentheses

Chiusa

#1796 aperta il 19 apr 2022

Vedi su GitHubGuida contributor
 (7 commenti) (2 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

There's a certain type of errors that can be hard to spot and will work fine. Since it's valid to do operations like || in this context is it even possible to have a rule that could notice when you might have meant the other one?

Fail

This is what you wrote

str.startsWith('str' || str.startsWith('str'))

Pass

This is what you meant

str.startsWith('str') || str.startsWith('str')

Additional Info

No response

Guida contributor

Direzione di ricerca
Indaga i pattern delle regole esistenti nel repository eslint plugin unicorn (ad esempio, come controllano i nodi AST). La regola deve rilevare parentesi posizionate erroneamente attorno agli operatori logici all'interno delle chiamate di funzione. Esamina la regola 'no unnecessary parens' e regole simili basate su AST. Considera l'uso del visitor `CallExpression` per controllare gli argomenti che sono espressioni logiche (||, &&) senza un raggruppamento appropriato. Il problema mette in dubbio la fattibilità; potrebbe essere necessario l'input del maintainer per confermare l'approccio.
Tech stack
javascripttypescript
Dominio
developer experiencetooling
Tipo issue
Funzionalità
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ESLint rule developmentAST traversal
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50