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!