flaviojmendes / trilhainfo

https://trilha.info
Apache License 2.0
792 stars 76 forks source link

Adicionar eslint para contribuições #197

Closed nicholascostadev closed 1 year ago

nicholascostadev commented 1 year ago

Eslint

Seria legal ter um eslint pra manter um padrão no código, acabei vendo muita má formatação e seria facilmente resolvido com o eslint configurado.

Qualquer uma das configurações abaixo dão para configurar da maneira que gostar, não precisa ser a favor 100% de cada uma. Caso goste de uma delas, mas tenha alguns contras, é só ir adicionando nas rules do eslint o que não gostar e ir removendo.

Opções de padrões:

  1. Rocketseat. Pra usar basta estender com: "@rocketseat/eslint-config/react".
  2. Escolha através do próprio npx eslint --init respondendo as perguntas a partir do seu gosto (não possui todos plugins pro react que o da rocketseat tem).
  3. Padrão do próprio eslint dentro das opções dadas pelo comando: npx eslint --init (não possui todos plugins pro react que o da rocketseat tem).
julianosirtori commented 1 year ago

Estou trabalhando nisso

nicholascostadev commented 1 year ago

Dá uma geral no que você tá pensando em usar, ele deu a ideia também de fazer uma votação pra galera dizer qual seria a melhor na opinião do chat

julianosirtori commented 1 year ago

Eu acabei de enviar a PR #202 Acabei utilizando o eslint com stantard e do @rocketseat/eslint-config/react Tambem adicionei o husky para rodar sempre um lint ao fazer o commit.

Achei arriscado rodar o eslint em todos os arquivos, tem muita coisa para mudar, como adicionar o "key" nos [array].map

Acho mais seguro ir corrigindo os arquivos aos poucos na medida q a gente for editando eles.

nicholascostadev commented 1 year ago

Verdade, pode ser melhor deixar da maneira que tá mesmo e ir arrumando o que for commitando aos poucos.

Gostei da maneira que vc fez, agora fica a critério do flavio aí 👍