Lint, test, format workflow e pre-commit hook

[mstrk]

Olá,

• Modificado o main workflow para só fazer trigger quando um pull request é open, edited ou reopen, também foi renomeado para lint-and-test.yml;
• Adicionado um pre-commit hook com husky que é instalado no script prepare que corre após o comando install;
• prettier movido para o pre-commit hook;
• Adicionado uma base para testes com jest para já ter incluido no workflow mas pode ser mudado facilmente após uma decisão no Adicionar testes #21;

[gitpod-io]

[IvoPereira]

Parece-me de longe a melhor solução para o problema.

Reparei que não colocaste o prettier na pipeline. Foi por alguma razão em específico? Senão tiver sido, se puderes adiciona e dou merge.

Obrigado pela contribuição!

[mstrk]

@IvoPereira se reparares na linha 2 do .eslintrc.json já estás a estender as rules do prettier e na linha 5 forças o error, penso que o default seja warning, o que faz com que nenhum código possa ser merged mal formatado sendo que o script do lint vai falhar na ci e on pre-commit.

Só fazemos uso do prettier no pre-commit hook para formatar código antes de correr o lint script, assim é menos frustrante para quem não usa eslint no seu IDE e não vir os erros de lint on development time.

[IvoPereira]

Bem-visto, escapou-me esse pormenor 👍

Lgtm