clutchski/coffeelint

add rule that if(!foo) should be replaced with unless

Open

#381 aperta il 18 feb 2015

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)CoffeeScript (169 fork)batch import
Help Wantednew rule

Metriche repository

Star
 (1203 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

basically - if you have an if block with no else, and you have if not or if !blah, it should be replaced with unless.

if !user
    return res.badRequest ...

should become

unless user
    return res.badRequest..

Guida contributor

Direzione di ricerca
Esamina le regole esistenti in coffeelint, come la regola 'no unnecessary double parens', per capire come aggiungere una nuova regola. Guarda i file sorgente nella directory 'src', in particolare 'rules/' per vedere la corrispondenza di pattern sui nodi AST. La richiesta è per una regola che trasformi le condizioni 'if not' o 'if !' in 'unless' quando non c'è una clausola else. L'implementazione probabilmente comporta il controllo dell'AST per i nodi 'If' e poi suggerire la sostituzione.
Tech stack
Nessuno
Dominio
toolingdeveloper experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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
GitCoffeeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60