Practices of good code review, or what is code review in 15 minutes. Nikita Sobolev's talk at DUMP in Kazan

In 2019, Nikita Sobolev , CTO of wemake.services, spoke at DUMP in Kazan . And Nikita spent almost 40 minutes trying to boil the brains of the Backend section listeners by talking about code review. Today we want to give a transcript of this "explosive" report, so that if the brains are seething, then all at once.

And here, by the way, Nikita Sobolev himself during his speech.

.

, . : « 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 .