良好的代码审查惯例,或者什么是15分钟内的代码审查。Nikita Sobolev在喀山DUMP的演讲

在2019年,wemake.services的首席技术官Nikita Sobolev在喀山的DUMP发表演讲尼基塔(Nikita)在将近40分钟的时间里试图让后端部分的听众大吃一惊,他们在谈论代码审查。今天,我们想提供一份这份“爆炸性”报告的笔录,以便如果大脑沸腾了,那么一下子就可以了。



顺便说一下,在这里,尼基塔·索博列夫(Nikita Sobolev)自己在演讲中。







.

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






All Articles