Код-ревью с уважением - Ангелина Купцова «КОРУС Консалтинг»

Ангелина Купцова, старший разработчик департамента аналитических решений ГК «КОРУС Консалтинг», на конференции HolyJS 2024 Spring рассказывает о код-ревью - сильном инструменте, который может как созидать, так и разрушать, поэтому важно научиться пользоваться им правильно.
Что может помочь вырасти разработчику или полностью убить его мотивацию? Что может усилить командную работу или полностью разрушить коллектив? Что может улучшить проект или превратить его в груду хлама, которую невозможно поддерживать? Ответ на все эти вопросы - код-ревью.
Больше информации и проектов на сайте data.korusconsulting.ru/
#javascript #frontend #корусконсалтинг

[@N5O1]

чушь какая-то, словно для детей в садике рассказывает. кодревью - это (да ладно) обзор кода и кодревью несет исключительно рекомендательный характер, кроме тех случаев, когда нейминг или форматирование идет в разрез с установленными для всех правилами, которые обычно где-то должен быть задокументированны. баги проверчют автотеств и куа, форматирование и нейминг из документации, остается только подход к проектированию. если во время код реаью возникает какое-то недопонимание или обсуждение, то тема сразв же закрввается и уезжает в общий чат в слак(или где вы общаетесь), а от туда уже или забивается на изначальную рекомендацию или код уходит на доработку. вот и весь кодревью

[@N5O1]

6:50 я бы сказал что код ревью это вообще не про баги, это про проверку кодстайла и подходов к написанию кода (архитектура, паттернв, алгоритмы и тд), зачем проверять код на баги, если это делпют автотесты, или зачем запускать код и смотреть как он рпьотает, если человек который разработал фичу - разработал и выпонил задание с борбы, а не просто из головы код накидал. код ренью это про однооьразие кода, его читабельность/пончтность и последующую поддержку. зачем нужен человек в команде, который неизапускает свой код, не читает задачи, не запускает тесты и кладет на линтер? эти вещи сами собой разумеющиеся это на кодревью не должно проверяться