Volta a usar a biblioteca remove-markdown
#1843
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| options.htmlTagsToSkip = Object.hasOwn(options, 'htmlTagsToSkip') ? options.htmlTagsToSkip : []; | ||
|
|
||
| var output = md || ''; | ||
| options.throwError = Object.hasOwn(options, 'logError') ? options.throwError : true; |
There was a problem hiding this comment.
Estamos usando logError em algum lugar? Não entendi essa linha.
There was a problem hiding this comment.
Foi só uma forma de passar a throwError como true para a remove-markdown pra gente registrar via API do Axiom se algum erro ocorrer. Como hoje queremos isso sempre, poderia muito bem ser só options.throwError = true.
Mas já deixei de uma maneira que, se quisermos usar a customRemoveMarkdown em algum lugar que não faça sentido usar a API do Axiom, a gente pode simplesmente desativar esse log passando logError = false. E se, no futuro, algum caso exigir algo mais elaborado (talvez logar o erro, mas sem passar pelo pino), essa linha pode ajudar a entender por onde começar a mudar o comportamento desse log. Certamente dá pra fazer isso de uma maneira melhor, mas ficar pensando nisso seria ainda mais overengineering do que já é o que eu fiz. 😅
|
Ontem, pra conseguir fazer o deploy em homologação, precisei editar um comentário existente, pois a nova versão da
|
Mudanças realizadas
A biblioteca
remove-markdownpassou por um bom período sem manutenção, por isso fizemos melhorias no código dela e trouxemos para dentro do nosso repositório.Só que agora a biblioteca voltou a ser mantida e já possui as melhorias que fizemos localmente, além de outras melhorias, então podemos voltar a usá-la.
Aproveitei para melhorar a performance do nosso
trimcustomizado, adicionando testes que não passariam com a versão anterior.Tipo de mudança
Checklist: