Faça a diferença com Code Review

Como implementar a cultura de code Review de forma eficiente, quais são os pré requisitos, motivações, benefícios, exemplos de o que verificar em um Code Review, prática do dia a dia utilizando Code Review, que hora do dia, quanto tempo em média vamos fazer, qual o tamanho ideal de um Pull Request, como eu descrevo o Pull Request, devemos seguir um checklist, sugestões de boas práticas, exemplo de ferramentas para agilizar o processo, qual a importância, como é o processo de adoção, o que fazer, e não fazer, como se comportar, como é a produtividade se aplicado corretamente, e os efeitos colaterais das disfunções, TDC 2018, Trilha XP,

Transcript

2. Faça a diferença com Code Review Rafael M. Sienna
3. O que é ?
4. Pré-requisitos ◦ Controle de versão ◦ Code Style ◦ Cultura da Empresa
5. Alinhamento das práticas • KISS • DRY • YAGNI • SOLID Motivações
6. Jardinagem Motivações
7. Assertividade do Code Style Motivações
8. Melhoria contínua Motivações
9. Benefícios ◦ Bugs são encontrados, ◦ Propaga a responsabilidade do código, ◦ Troca de conhecimento e idéias ◦ Aumento do “bus factor” ◦ Promove e evolui padrões em toda a organização ◦ Mentoria de desenvolvedores ◦ Melhor integração com o time ◦ Cultura de feedback
10. PULL REQUEST
11. • Explique o quê e por quê • Devo seguir um checklist • Qual o tamanho ideal • O foco é a transparência • Posso colocar um exemplo • Quantos revisores • Quem deve ser os revisores Criar Pull Request
12. • Quando revisar, breve • Quanto tempo gastar, 1h • Devo seguir um checklist • Se tem itens repetidos melhor comentar no início • Comentar na linha do código • Muitas dúvidas • Respostas positivas • Devo recusar o PR • Sempre a mesma pessoa Revisar Pull Request
13. Revisar o que ? ◦ Atende os requisitos da história ◦ Code Style ◦ Atende os casos de sucesso e falha ◦ Legibilidade ◦ Arquitetura e design ◦ Performance ◦ Erros ◦ Falhas de segurança
14. • Camaradagem • PR Atômico • Não seja hater/flooder • Linguagem agressiva • Briga pela paternidade Não fazer
15. Ferramentas & Automatização ◦ Ferramentas integradas ao Pull Request • CircleCI, Hound CI • Code Climate, Codacy, gitscrum
16. NÓS MOVEMOS O MUNDO. Centro Av. Presidente Wilson, 231 29º andar (21) 2240-2030 Cidade Monções Av. Nações Unidas, 11.541 3º andar (11) 4119-0449 Savassi Av. Getúlio Vargas, 671 Sala 800 - 8º andar (31) 3360-8900 Ilha do Leite Rua Sen. José Henrique, 199 2º andar (81) 3018-6299 WWW.CONCRETE.COM.BR @rmsphd Linkedin rafaelsienna
17. Referências ◦ https://blog.jetbrains.com/upsource/2016/01/25/what-to-look-for-in-a-code-review-book/ ◦ https://github.com/getquill/quill/pull/322 ◦ https://medium.com/humans-create-software/code-review-discipline-and-working-contracts- cee784be8e55 ◦ http://guidelines.plataformatec.com.br/pull-requests.html ◦ https://github.com/thoughtbot/guides/tree/master/code-review ◦ https://www.slideshare.net/RaquelPau/code-review-68177398 ◦ http://chris.beams.io/posts/git-commit/