Bonnes pratiques de révision de code, ou ce qu'est une révision de code en 15 minutes. Conférence de Nikita Sobolev à DUMP à Kazan

En 2019, Nikita Sobolev , CTO de wemake.services, a pris la parole à DUMP à Kazan . Et pendant près de 40 minutes, Nikita a essayé de faire bouillir le cerveau des auditeurs de la section Backend, en parlant de révision de code. Aujourd'hui, nous voulons donner une transcription de ce rapport "explosif", de sorte que si les cerveaux bouillonnent, alors tout à la fois.

Et ici, au fait, Nikita Sobolev lui-même lors de son discours.

.

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