edgurgel/solid

break and continue with whitespace control issue

Aperta

#86 aperta il 6 set 2021

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Elixir (56 fork)github user discovery
bughacktoberfest

Metriche repository

Star
 (257 stelle)
Metriche merge PR
 (Merge medio 17g 10h) (2 PR mergiate in 30 g)

Descrizione

     Render result was different!
     Input:
     ########################

     this should print

     {%- break -%}

     this should not print

     ########################


     code:  liquid_output == solid_output
     left:  "########################\n\nthis should print\n"
     right: "########################\n\nthis should print"

     Render result was different!
     Input:
     ########################

     this should print

     {%- continue %}

     this should not print

     ########################


     code:  liquid_output == solid_output
     left:  "########################\n\nthis should print\n"
     right: "########################\n\nthis should print"

https://github.com/edgurgel/solid/blob/master/test/integration/whitespace_control_cases/break_tag_test.exs#L3-L15 & https://github.com/edgurgel/solid/blob/master/test/integration/whitespace_control_cases/continue_tag_test.exs#L3-L15

Guida contributor

Direzione di ricerca
Esamina la logica di controllo degli spazi bianchi per i tag break e continue nel parser. Verifica come viene applicata la rimozione degli spazi e assicurati che si comporti in modo coerente con altri tag simili.
Tech stack
elixir
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
ElixirGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80