Fluxo de Trabalho de Code Review e Status de Comentários

Este documento estabelece o padrão para o ciclo de vida de Pull Requests (PRs) e o fluxo de status dos comentários dentro do Azure DevOps. O objetivo é garantir que nenhuma correção seja perdida e manter a clareza sobre quem deve atuar em cada etapa.

Definições de Papéis

• Autor: Desenvolvedor responsável pela implementação e abertura do PR.
• Revisor: Desenvolvedor responsável pela análise técnica.

O Fluxo do Pull Request

1. Abertura do PR

O Autor deve abrir o Pull Request apenas após o cumprimento dos seguintes requisitos:

• Implementação do caso finalizada.
• Funcionalidade validada e aprovada pela equipe de QA/Testes.

2. Revisão Inicial (Status: Active)

O Revisor analisa o código. Ao encontrar pontos de atenção, bugs ou sugestões de melhoria, insere comentários nas linhas correspondentes.

• Ação do Sistema: O Azure DevOps define automaticamente o status do comentário como Active.
• Significado: O ponto requer ação ou resposta do Autor.

3. Tratativa e Correção (Status: Pending)

O Autor atua sobre os comentários do Revisor.

• Após realizar o commit com a correção solicitada ou responder a uma dúvida, o Autor deve alterar manualmente o status do comentário para Pending.
• Significado: A correção foi realizada e aguarda revalidação do Revisor.
• Nota: O Autor não deve marcar o comentário como Resolved.

4. Validação e Encerramento (Status: Resolved)

O Revisor é notificado das atualizações e verifica os itens marcados como Pending.

• Se a correção estiver satisfatória, o Revisor altera o status para Resolved.
• Significado: O tópico foi concluído e o comentário será colapsado/ocultado da visualização principal.
• Se a correção não for satisfatória, o Revisor pode reverter o status para Active e adicionar novas observações.

Boas Práticas

1. Não resolva seus próprios tópicos: Apenas quem criou o comentário (ou outro Revisor) deve marcá-lo como Resolved. Isso garante o duplo cheque de qualidade.
2. Use o status "WontFix" com cautela: Caso uma sugestão não possa ser aplicada, discuta no chat do PR antes de marcar como "Won't Fix".
3. Contexto nos comentários: Ao passar para Pending, se a correção não for óbvia, adicione uma resposta explicando o que foi feito (ex: "Corrigido no commit x").