Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Code Review

Sponsored · Ship Features Fearlessly Turn features on and off without deploys. Used by thousands of Ruby developers.

Code Review

Avatar for Rafael Pazini

Rafael Pazini

January 23, 2018
Tweet

More Decks by Rafael Pazini

Other Decks in Programming

Transcript

  1. What is it? Code Review é um momento para discutir

    e debater o código que foi implementado e que será propriedade do time a partir daquele momento.
  2. Why is it important? • Conseguimos prevenir alguns erros básicos;

    • Entendemos o que está mudando no projeto; • Podemos debater sobre formas de implementação; • Aprendemos com códigos de outras pessoas; • Refletimos sobre o valor de uma crítica construtiva.
  3. • Eu consigo entender este código? • Existe algum erro

    óbvio de lógica? • Olhando para os requisitos, todos os casos são totalmente implementados? • A cobertura de testes é suficiente para a nova implementação? • O novo código está coerente com os guidelines existentes? Some questions to do during review
  4. Some things you don't discuss • Coisas que podem ser

    corrigidas com processos automatizados (formatação de código)
  5. Antes de começar Tenha em mente que o code review

    faz parte do seu trabalho Separe um tempo para fazer isto com calma E por fim… Pegue um café ;D
  6. E se... as linhas de código que você escreveu as

    vidas de pessoas que você mudou? ==