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 

 

 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. 

 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". 

 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").