-
-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adicionados os termos tag, loop e xml #151
Conversation
Signed-off-by: George Araujo <george.gcac@gmail.com>
✅ Deploy Preview for diciotech ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Uma dúvida @levxyca, eu adicionei
Inclusive acho que essa discussão seria legal ter e botar o resultado dela em algum lugar como boas práticas pra caso algo do tipo aconteça de novo. |
Perfeita observação @george-gca e muito importante! Abri uma discussão aqui em #188 (comment) para podermos decidir sobre isso, após termos uma decisões, podemos adicionar ela em um documento de boas práticas sim 🚀 |
Inclusive eu fui fazer o merge com a main agora (e fiz tudo pela interface do GitHub mesmo) e pensei, e se eu errar no json, por exemplo esquecer uma vírgula entre os itens? As ações do netlify detectam isso? Ou será que seria melhor adicionar alguma ação de validação do json com os termos pra rodar nas PRs tipo json-yaml-validate? |
@george-gca boa questão! normalmente o que eu faço é verificar se ocorreu tudo certo com o deploy pelo netlify e caso veja que o site tá estranho (normalmente sem os cards) eu vou investigar e em 99% dos casos é alguma vírgula no json 😅 Mas pensando aqui, ter algo desse tipo como o json-yaml-validate ajudaria a evitar esse trabalho, assim que a pessoa abrir a PR ela já vai saber que tem esse problema né 🤔 |
Vou ver se consigo montar esse rapidinho e fazer um teste no meu fork, qualquer coisa mando um PR. |
@george-gca perfeito!!! agradeço demais! |
@george-gca para andarmos com esse PR no momento, enquanto não temos uma decisões, o que acha fazermos como uma das sugestões e colocar, por exemplo, "tag em html/xml"? |
Feito. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Obrigada pela contribuição 🫰🏻🫱🏻🫲🏻
No description provided.