Revisão

avaliação formal de algo com a intenção de promover mudanças, se necessário.

Code Review

Análise do código fonte com intuito de achar defeitos:

• Segurança, Funcional, Performance...
• Complementa a análise estática.
• Detecção antecipada.

3 Regras para um REview

Regra 1

O reviewer não pode ser quem escreve o código.

• Se somos capazes de achar bugs no código, somos capaz de evitá-los.
• Análise Parcial.
• O reviewer deve ter uma perspectiva diferente do desenvolvedor.
• O reviewer (preferencialmente) deve participar de outro projeto.

Regra 2

O reviewer deve entender a linguagem sendo revisada.

Regra 3

Foco no Objetivo: segurança, performance, funcionalidade, etc. 

Não deve basear-se em preferências pessoais.

• Os devs tem coisas mais interessantes para fazer.
• Os devs tem mais o que fazer.
• Os devs tem deadlines agressivos e a revisão acaba facilmente sendo desconsiderado.
• Os devs não gostam do código alheio.
• Os devs não sabem O QUE revisar. 

A grande Pergunta

O que revisar?

Não deixem os devs escolherem o que revisar arbitrariamente.

Code review são para Reviewers

• Utilize uma ferramenta para gerenciar o que é associado para cada reviewers.
• Cada reviewer tem uma fila de revisões a serem feitas.
• 1 revisão = 1 Pull Request

garanta

• Cobertura de código - Todo o código deve ser revisado.
• Responsabilidade - O desenvolvedor tem uma tarefa publicamente associada a ele.
• Entregáveis - Crie evidências, aumente a motivação para revisar.

Differential, code review
Diffusion, Navegador de repositorios 

Phriction, bug tracker
Maniphest, wiki

Entre outras ferramentas. . .

Comunicação Persistente

Discussões que envolvem mudanças nas regras de negócio ou implementações mais espertas que necessitam de algum tipo de explanação.

Design Review

É muito frustrante descobrir ao fim de uma implementação que o design é ruim e ter que troca-lo.

Até mesmo deixar o design ruim e não escalavel, com isso nossa code-base piora inegavelmente, pois o custo de manutenção aumenta exponencialmente com o tempo.

Notificações 

Facilita o gerenciamento de tarefas, quando um desenvolvedor faz um commit ou alteração de status em discussão técnica os interessados podem receber as notificações em tempo real para agilizar o processo de troca de informações.

Post-Commmit

Ferramenta facilita a atividade de code review tardio.
O que não deveria acontecer, mas vem se tornando comum, ou pela falta de capacitados para a tarefa de code review ou pela urgência de algumas entregas.

Calendario de Reviews

Para agendamento de tarefas como code-review, design review e outras.
Mas o calendar não server? Serve! Vale a pena misturar tarefas de desenvolvimentos e reuniões com ela?

Markup

Traz a capacidade de colocar marcações inteligentes que interligam commits, code reviews, design reviews e discussões. Esse tipo de marcação enriquece e agiliza as discussões.

Não é mais necessário ter que lembrar de quando o email foi mandado se as discussões forem feitas nesse tipo de ferramenta.

É possivel tabém fazer formatação de código para exemplos dentro do texto.

Tasks

Atribuição de tarefas. e o JIRA?

Talvez não seja a ferramenta correta por exemplo para montar essa apresentação. Tarefas de desenvolvimento em geral.

Web Design Review

Ferramenta para revisão de mocks, orientada principalmente por imagens do prototipo.

Feed de Noticias

Transparência para tudo que acontece em todos os projetos.