Code review é uma prática de revisão de código muito utilizada em projetos open-source ou em projetos empresariais. Ela consiste em os membros do time revisarem o código de um colega antes de integrá-lo à base de código.
Pull request é quando você envia uma sugestão de melhoria para o repositório.
- Ao criar um PR certifique-se se ele segue a seguinte descrição:
* Funcionalidade: <Funcionalidade que você está adicionando>
* Card: <Link do card no Trello que corresponde a essa tarefa>
* Descrição: <curta descrição do que foi feito>
- Verificar você mesmo o código em busca de falhas de padronização e erros
- GIT
- Todos os commits são pequenos e divididos logicamente
- Os commits têm mensagens curtas e descritivas
- O commit está livre de arquivos de IDE/editor e afins
Itens que você deve checar antes de abrir o PR ou ao revisar o PR de outro membro
- O código está funcionando?
- O layout ou especificação está de acordo com o definido?
- Você testou o que desenvolveu?
- Você verificou se não quebrou outra parte da aplicação?
- O seu código está seguindo os padrões de organização do projeto?
- O código está repetitivo?
- O nome de variáveis, funções, arquivos, classes, métodos... está de acordo com os padrões de projeto?
- Você está separando as responsabilidades do código de maneira adequada?
- A branch do PR está sincronizada com a de destino e livre de conflitos?
- Certifique-se que você está evitando valores "hardcoded" e está separando os valores em arquivos para constantes, strings...
- Você está evitando ternários e blocos muito longos de
if/else? - Adiciona comentários quando necessário, descrevendo o porquê?
- Seu código está livre de itens para depuração, como logs desnecessários?
- O código está formatado de acordo com as diretrizes do projeto?
- Segurança
- Usabilidade