Buenas prácticas de revisión de código, o qué es una revisión de código en 15 minutos. Charla de Nikita Sobolev en DUMP en Kazán

En 2019, Nikita Sobolev , CTO de wemake.services, habló en DUMP en Kazán . Y Nikita pasó casi 40 minutos tratando de hervir el cerebro de los oyentes de la sección Backend hablando sobre la revisión de código. Hoy queremos dar una transcripción de este informe "explosivo", para que si los cerebros están hirviendo, entonces todos a la vez.

Y aquí, por cierto, el propio Nikita Sobolev durante su 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 .