Skip to content

Instantly share code, notes, and snippets.

@henriquejensen
Last active March 27, 2023 22:54
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 2 You must be signed in to fork a gist
  • Save henriquejensen/907b27722a658e0b34272dafb71be63d to your computer and use it in GitHub Desktop.
Save henriquejensen/907b27722a658e0b34272dafb71be63d to your computer and use it in GitHub Desktop.

Boas práticas para Code Review

  • os testes estão passando
  • não existem erros de lint
  • commits estão divididos (não existe o commit bomba), e eles possuem uma boa mensagem
  • variáveis possuem nomes que fazem sentido (evitar nomes simples como const a, const minhaVar, etc.)
  • Os nomes dos componentes e das funções são coerentes (traduzem o que aquele componente e aquela função se pretendem a cumprir).
  • a lógica esta entendível (código é feito para um humano)
  • esta usando os métodos do javascript para array (map, filter, reduce) ao invés de for e while
  • Se tiver comentários, eles são pontuais e em partes relevantes
  • Esta utilizando tags semânticas
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment