O documento aborda a evolução do processo de revisão de código, destacando a importância de práticas como rubberduck debugging e a utilização de ferramentas como Rubocop para melhorar a qualidade do código. Inicialmente, as revisões focaram em aspectos superficiais, mas com o tempo passaram a considerar fatores como arquitetura e segurança. Resultados positivos foram observados em um ano, com maior difusão de conhecimento e colaboração na equipe.

1. Code Review + Rubocop
Ferramentas, mentalidade
e evolução do processo

2. lucas.uyezu@locaweb.com.br
Lucas Uyezu
Dedicado -> IaaS -> Eng. Produto

3. Motivadores

4. ● Diluir conhecimento técnico e de negócios
● Rubberduck debugging
● 1 gem, técnica ou manha explicada num MR
ensina a todos

5. Rubberduck debugging
https://en.wikipedia.org/wiki/Rubber_duck_debugging

6. Rubberduck debugging

7. Evolução

8. 1o. nível

9. Primeiros code reviews focaram em:
● tabs ao invés de espaços
● alinhamento
● if ao invés de unless

10. Não há impacto/valor real no produto

11. 2o. nível

12. Com o tempo, passamos
naturalmente a focar em:
● sanity check
● arquitetura
● efeitos colaterais
● performance
● segurança
http://blog.fogcreek.com/effective-code-reviews-9-tips-from-a-converted-skeptic/

13. Arquitetura

14. Performance

15. Segurança

16. https://github.com/bbatsov/rubocop

17. 4 tipos principais de cops

18. LINT
Possíveis erros e más práticas:
● useless_assignment
● debugger/pry

19. METRIC
Métricas (configuráveis) para se
aplicar ao código:
● Métrica ABC
● Complexidade Perceptível

20. http://c2.com/cgi/wiki?AbcMetric

21. RAILS
Cops exclusivo para RoR:
● delegate
● scope_args: scopes que não
estão encapsulados em
procs/lambdas

22. scope_args

23. scope_args

24. Style
Cops de estilo (duh)
● alinhamento de hash com n linhas
● Indentação

25. Plugins para Vim, Emacs,
Sublime & TextMate

26. ● Todos os projetos usam o mesmo .rubocop.
yml
● Antes do commit, ele passa o rubocop nos
arquivos do diff.
● Se tiver erros, não rola commit.

27. Resultados
(Após 1+ ano)

28. ● Conhecimento difundido: quase não existem
áreas em IaaS com 'pai'
● Cloud+Dedicado+Load Balancer + Becape +
Painel de IaaS + Client VLAN: 9 PRBs + 3
frozen

29. Moral da história

30. Muita gente ocupada,
atribuir um Deadline ao
MR e seguir em frente

31. Projetos antigos

32. rubocop -lR

33. Antes do "pente-fino"

34. Não seja xiita
(rubocop não passava no
nosso rubocop.yml)

35. Alarmar é mais
importante do que passar
tudo e ganhar estrela de
bom menino

36. Dúvidas?

37. Obrigado!