jfmengels/elm-review-simplify

Simplify `if x then (_, True) else (_,False)` (and other conditionals that differ only by a boolean literal)

Open

#11 aperta il 17 lug 2021

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Elm (10 fork)github user discovery
enhancementhelp wanted

Metriche repository

Star
 (22 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Haven't put a ton of thought into how difficult it would be to implement as a rule, but I ran into it in my code today and noticed it wasn't caught by elm-review-simplify currently.

if bar then
    ( foo, True )
else
    ( foo, False )

can be simplified to just:

( foo, bar )

Honestly this could be extended to basically any two expressions that differ only by a Boolean between conditions, but it might be challenging to check for that.

E.g.

if bar then
    { a = foo, b = True }
else
    { a = foo, b = False }

if bar then
    func foo True
else
    func foo False

Off the top of my head, one could maybe descend into the two Node Expressions of an IfBlock and check for the presence of a boolean literal, then compare the ASTs for equality with the literal swapped?

Guida contributor

Direzione di ricerca
Studia il modulo AST di elm review per capire come sono rappresentate le espressioni. Cerca regole esistenti che attraversano e confrontano nodi AST. Implementa una regola che rileva blocchi if else in cui i due rami differiscono solo per un letterale booleano nella stessa posizione, quindi semplifica sostituendo il condizionale con l'espressione comune e il booleano dalla condizione.
Tech stack
Nessuno
Dominio
developer 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
Elmelm reviewAST manipulation
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60