Code review do jeito certo

Code review do jeito certo

Nesta palestra vamos falar um pouco sobre o que é Code Review e como podemos melhorar cada vez o processo de revisão para que seu time e seu código não te odeiem

0a8f40c6cc23fb6c8aad4de732c5abaf?s=128

Lucas Santos

May 11, 2019
Tweet

Transcript

  1. 6.

    /khaosdoctor @_staticvoid lsantos.dev @khaosdoctor Por que devemos fazer isso? Não

    é só porque é legal - Garantir um melhor design de código - Garantir um código mais livre de bugs - Nos livrar de Edge cases - Compartilhar conhecimento com o time - Distribuir responsabilidades
  2. 9.

    /khaosdoctor @_staticvoid lsantos.dev @khaosdoctor 2. Veja pontos relevantes - Arquitetura

    em níveis mais altos - Edge cases e escopos que podem não ter sido cobertos - Cobertura de testes - Nomenclatura e estilo
  3. 15.

    /khaosdoctor @_staticvoid lsantos.dev @khaosdoctor 4. Dê seu feedback Sempre é

    um problema de comunicação - Encoraje colaboração e discussão - Não corrija você mesmo - Encoraje a ideia que feedbacks são bons! - "Isto está ótimo! Vamos deixar ainda melhor" - Comunicação não Violenta - Prefira "Nós" a "Você"
  4. 17.

    /khaosdoctor @_staticvoid lsantos.dev @khaosdoctor Melhore ainda mais seu feedback -

    Sempre comece com fatos - Dê referências externas - Nunca seja imperativo - Sempre convide o autor para uma discussão
  5. 18.

    /khaosdoctor @_staticvoid lsantos.dev @khaosdoctor Problemas comuns Releve algumas coisas, você

    também já esteve lá - Desconhecer padrões do time - Familiaridade com o processo de CR - Falta de entendimento do negócio - Pouco conhecimento técnico - Ego - Gírias e piadas internas - Pressão externa
  6. 25.

    /khaosdoctor @_staticvoid lsantos.dev @khaosdoctor Outros problemas com PR's - Não

    há um template - Sem referências a issues ou outras PR's
  7. 27.

    /khaosdoctor @_staticvoid lsantos.dev @khaosdoctor Como fazer uma má code review

    Todo mundo vai adorar! 1. Não avalie o contexto, pule direto para a revisão 2. Nunca peça ajuda, afinal, você sabe de todas as coisas 3. Comente tudo que ferramentas de análise já identificaram 4. Perca tempo discutindo a mesma coisa repetidamente 5. Se prenda a estilos
  8. 37.

    /khaosdoctor @_staticvoid lsantos.dev @khaosdoctor Por que code review? - Melhora

    as estimativas do time - Permitem que desenvolvedores espaireçam - Geram mentoria para novatos - Geram Peer Pressure, que geram melhores desenvolvimentos
  9. 39.

    <refs> /khaosdoctor @_staticvoid lsantos.dev @khaosdoctor - Code Review, comunicação amigável

    e cultura - http://bit.ly/2HPWK8Y - Code Reviews: How can we do it better? - http://bit.ly/2HNFEZb - Code Reviews in Atlassian - http://bit.ly/2HR5The - Code Reviewing like a champion - http://bit.ly/2HQ80lB - Better Code Reviews - http://bettercode.reviews - Qualidade além do código pt. 1 - http://bit.ly/2WC0JJy - Qualidade além do código pt. 2 - http://bit.ly/2WCSA7p - Usando Pull Requests a favor do time - http://bit.ly/2WA82RU - 7 Masters: Qualidade além do código - http://bit.ly/2WA8bEW