Quando se trata das "alterações" feitas neste commit, PR ou MR, é fácil deixar comentários dizendo se aquela mudança está boa ou não. Mas perceber e comentar as partes que deveriam ter sido alteradas e não foram continua parecendo bastante difícil para mim, mesmo depois de fazer code review há bastante tempo. Isso é ainda mais verdadeiro quando não se trata de uma base de código com a qual eu tenha muita familiaridade.
Consigo pensar em alguns métodos, mas seria bom ter mais mecanismos. Queria saber como vocês têm lidado com isso.
- Ao implementar: estruturar de forma que as partes que precisariam ser alteradas junto no futuro também possam ser cobertas pelos testes.
- Ao escrever o PR: adicionar no template do PR perguntas adequadas para o reviewee considerar. (e.g., existe algum PR anterior com objetivo semelhante a este? Comparando com aquele PR, será que não ficou faltando alguma coisa neste PR?)
- Sempre que acontecer algo que "deveria ter sido feito, mas foi esquecido" nessa base de código, registrar isso e, a cada vez, atualizar o template do PR com esse conteúdo como exemplo.
- Ao revisar o PR: adicionar ao guia de revisão orientações para que o revisor procure partes faltantes de forma semelhante às perguntas acima.
Ainda não há comentários.