Implementa Github Actions para linting de commit

[filipedeschamps]

Closes #42 (issue relacionada a atual Milestone)

Usei uma Action feito pelo brasileiro @wagoid. Vamos ver o que vai acontecer 👍

O que aconteceu

Leia a thread, pois mostro o desenvolvimento e os prints, tudo funcionou como esperado.

Esse PR faz 3 coisas

1. Primeiro modifica a Action que faz o linting dos styles para especificar que ela é sobre styles.
2. Adiciona a Action que faz o linting dos commits.
3. Adiciona uma ferramenta para ajudar a construir mensagens de commits no padrão correto. Para acessar ela, após fazer o staging dos arquivos normalmente, basta rodar o comando npm run commit (não é obrigatório, é só para ajudar mesmo, porque não faz diferença commitar manualmente).

O que ese PR não faz

Como sugerido na issue da milestone, esse PR não adiciona o husky, não achei necessário no momento, dado que a garantia está no check do CI, e na correria da vida acho que é importante deixar a pessoa conseguir empurrar de qualquer forma para garantir que a cópia da sua alteração fique remota. Aí depois é só a pessoa consertar a mensagem.

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/tabnews/tabnews/B7M4RbsVG2zg1b4B8Rye8Nf4b6zj
✅ Preview: https://tabnews-git-commit-lint-tabnews.vercel.app

[filipedeschamps]

Ok, funcionou, agora vou mandar um commit que não respeita os padrões config-conventional e o PR deveria quebrar 👍

[filipedeschamps]

Novamente, falou com sucesso 😂 não deveria passar mesmo 👍

Agora vou consertar o commit usando o commitizen sugerido por @karanalpe 👍

[CarlosZiegler]

Muito legal, qual a consequência de não passar nesse lint? Ele não vai poder ser mergeado correto ?

[filipedeschamps]

Exato meu caro! Assim evitamos ter um histórico maluco no git, e como benefício de seguir esse padrão, poderemos no futuro gerar automaticamente changelogs 🤝

…

On Wed, Jul 21, 2021 at 10:47 PM Carlos Ricardo Ziegler < ***@***.***> wrote: Muito legal, qual a consequência de não passar nesse lint? Ele não vai poder ser mergeado correto ? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#80 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABANEEKLCD52255BZLZJYY3TY6WGHANCNFSM5AY5ILQA> .

[filipedeschamps]

Merged! Let's goooooo!!!

Em paralelo, vou refatorar a parte dos testes conforme sugerido pelo @huogerac nessa thread