Trabalhando com code review e pull request

Transcript

1. 1 Trabalhando com code review e pull request TDC ~ São Paulo ~ 2018
2. SOBRE NÓS Juliana Fernandes Software Developer Andriele Menegotto Software Developer
3. TRUNK BASED DEVELOPMENT ▪ Todos commitando direto na master ▪ É um dos princípios de Integração Contínua ▪ Após cada commit, o build pipeline é acionado: testando, deployando, testando de novo… ▪ Modelo de branching para desenvolvimento de software ▪ Feature Branches são branches que duram todo o tempo de desenvolvimento de cada funcionalidade ▪ Long-lived branches == merge hell
4. Contexto 4 ▪ Time distribuido(Porto Alegre, Santiago - Chile, Belo Horizonte) ▪ Desenvolvedores com diferentes experiências ▪ Muitas tecnologias no mesmo projeto ▪ Equipe de 20 desenvolvedores ▪ Projeto com domínio muito complexo ▪ Projeto com riscos de alto impacto
5. Contexto 5 ▪ TDD ▪ Pair programming ▪ Code review(on demand)
6. Porque mudamos? 6 ▪ Garantir mais qualidade nas entregas ▪ Consolidar uma visão técnica do time sobre a plataforma ▪ Impulsionar discussões técnicas ▪ Compartilhar conhecimento ▪ Impulsionar cultura Open Source
7. SHORT-LIVED BRANCH BASED DEVELOPMENT 7 ▪ Criado o Pull Request, convidamos a equipe a uma discussão, basicamente, o code review. ▪ Se necessário, a partir dos comentários, realizamos as mudanças. ▪ Depois das modificações, abrimos um Pull Request. ▪ Para qualquer modificação, criamos uma nova branch. ▪ Se o build dessa branch está verde, verificamos os approves. ▪ Fazemos merge no master.
8. Trabalhando com Pull Request 8 ▪ Não deixe os Pull Requests abertos por muito tempo, se a discussão ficar longa ou difícil, agende um dev-huddle. O code review não deve deixar a equipe bloqueada. ▪ Quanto maior o PR, mais difícil de revisar e pedir feedback. ▪ Escreva uma descrição clara e dê um contexto sobre as modificações • Referências externas ajudam!
9. Trabalhando com Pull Request 9
10. Trabalhando com Pull Request 10
11. Trabalhando com Pull Request
12. Trabalhando com Pull Request 12
13. DÚVIDAS? Juliana Fernandes jfernan@thoughtworks.com Andriele Menegotto amenegot@thoughtworks.com