Boas práticas de revisão de código, ou o que é revisão de código em 15 minutos. Palestra de Nikita Sobolev no DUMP em Kazan

Em 2019, Nikita Sobolev , CTO da wemake.services, falou no DUMP em Kazan . E por quase 40 minutos Nikita tentou ferver os cérebros dos ouvintes da seção Backend, falando sobre revisão de código. Hoje, queremos fazer uma transcrição desse relatório "explosivo", de modo que, se os cérebros estiverem fervendo, tudo de uma vez.

E aqui, aliás, o próprio Nikita Sobolev durante seu discurso.

.

, . : « code review. ». – code review, . ? :) , .

, , . ? – , . « ». – , . « ». , « code review» , , « code review». .

, - , , . : - ? – . . , . , , code review 15 . .

, code review?

, code review, . , . code review?

• - code review;
• ;
• , ;
• code review review;
• code review .

, , . , . , . : ? ? .

code review.

- , . ( , ) ( , ).

: . - , . , . , – . «» 15 2, 4 . , . ( code review ) Open-Source. ? -:

• Contributing.md – , ;
• Developer Docs – api- ;
• Architecture Decision Records – , . «» ;
• Wiki ;
• pull requests;
• – .

- , , Open-Source .

:

1. Gatsby.js – , ;
2. Dev.to – , , .. ;
3. Wemake-python-styleguide – , , « ».

— review . . ?

• . , review , , , , pull request. ? , .
  
  
• .
• .

: design review. Design review – , . , - . \ . , code review .

, , ! , .

(, , ). – . python.

• – importlinter. . , , . . ? , . , . . , - , layers. django_project. : urls, views, forms, models, logic. , logic – . Models logics. Forms models logic, .

• . , , . , .

• , «». , . , , . - , .. .

• , , – , – . api, – !

.

– « ».

code review , , . , . : .

, – .

- -, BDDSpec. , BDDSpec — , , . , . , BDD – , , . – , . , .

- , . «».

! . – Danger. :

2020 danger- : JS, Swift, Ruby, Kotlin Python. JS.

« CI» :

• Pull request , .
• Pull request - .

«» :

• Pull request .
• issue .

. , .

– bellybutton. ( ). , deprecated_fn(), . , , . , , . - , yaml :

, code review .

– . , , , – .

? -, . , , , . . -, « » . -, . , - , , . – , . : . : + . : « , ».

. :

• , ;
• , – 20 ;
• , – .

: Architecture on Demand. , . . .

last but not least – . , – code review, , , . :

• ;
• , ;
• .

: BDD ( , , ) Review Apps. « », . . , ZEIT GitLab. pull request .

, , , , , , , – , . , code review. .

. . , code review . . code review? Code review . . , , . , .

, :

1. ( 15 2., max – 4.);
2. review apps – ;
3. , , ;
4. , , – , ;
5. . , review .

, ? – , :

• ;
• ;
• .

code review. . « code review» «», 15 . : « , , ».

:)

code review?

P.S. DUMP` :) , DUMP 2020 20 .