Praktik peninjauan kode yang baik, atau peninjauan kode dalam 15 menit. Pembicaraan Nikita Sobolev di DUMP di Kazan

Pada 2019, Nikita Sobolev , CTO dari wemake.services, berbicara di DUMP di Kazan . Dan selama hampir 40 menit Nikita mencoba mendidihkan otak para pendengar bagian Backend, membicarakan tentang review kode. Hari ini kami ingin memberikan transkrip dari laporan "ledakan" ini, sehingga jika otak mendidih, maka semuanya sekaligus.

Dan di sini, ngomong-ngomong, Nikita Sobolev sendiri selama pidatonya.

.

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