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