Resultados diferentes na versão 2.0

[filipedeschamps]

@TulioAbreu vou escrever em português mesmo :)

Eu uso esse módulo para gerar o conteúdo final da minha Newsletter e eu inseri um bug ao atualizar as dependências e que comprometeu a edição dessa última sexta-feira. Esse bug não tem nada a ver com esse módulo aqui, mas no meio das atualizações de todas as dependências eu escrevi testes garantindo o comportamento entre o que era gerado antes de depois, e notei que o PR #3 faz o conteúdo mudar em alguns casos de forma estranha, olha só:

Esquerda (Original) vs Direita (Usando a V2)

Caso 1

Nota que após a atualização do módulo, ele adiciona um espaço a mais antes de fechar a tag.

Caso 2

Aqui ele está consumindo os espaços, mas também o caractere !

Caso 3

Ele remove dois espaços no início que deveria, mas um espaço ali no final depois do : que não deveria remover:

Diff inteiro

Você pode conferir aqui o diff inteiro (acessível por 1 mês): https://www.diffchecker.com/VHe6pxSv

Conclusão

O que acha de fazermos o revert do PR?

[TulioAbreu]

@filipedeschamps

Considerando que a versão introduziu bugs que aparentemente estão removendo parte do texto (o que eu considero algo crítico), acredito que o melhor por enquanto seria de fato um revert.

Você acha que seria válido alguma contribuição introduzindo testes nas etapas de parser, e em seguida uma possível resolução dos bugs? Dessa forma, futuras contribuições podem se beneficiar da certeza de que a solução de um bug não prejudicou algum outro caso de uso

[filipedeschamps]

Show @TulioAbreu ! Vou fazer o revert 👍

Sobre os testes, total!! Até porque isso também vai nos dar segurança para atualizar as dependências do projeto 👍