Привет! 🫂 🗣💭Предлагаю поговорить на тему необходимости code review в формате переписки и комментариев, а так же тех случаев, когда код ревьюит тим лид/тех лид.

🎤🎤🎤 Тема с Agile Days, и докладчик предлагает просто-напросто отказаться от code review именно в описанном выше формате, и предлагает перейти к одному из двух вариантов: каждый ревьюит за каждым или парное программирование. Так же, в докладе внимание уделяется важности словесной коммуникации, а не письменной.

У код ревью есть свои плюсы, которые можно поставить под сомнение (на самом ли деле эти плюсы существуют, и на самом ли деле это плюсы, или, скорее, минусы). И, так же, процессу code review сопутствуют проблемы, которые встречаются в процессе этого самого код ревью.

👩‍💻👩‍💻👩‍💻 К слову, с некоторыми проблемами я сталкивалась так же: - болезненное отношение к комментариям (один из кейсов описывала выше, когда разработчик написал в ответ на комментарий лида и запушил код с очень неконструктивным комментарием); - слышала и жалобы на несодержательные (и не совсем понятные почему комментатор прав, а разработчик, написавший код-нет) комментарии; - так же, одна из основных проблем на одном из проектов была как раз в большой "петле" код-комментарий-ответ-ответ на ответ. Особенно это заметно в командах, в которых код ревьюит человек из другой команды; - ну и, был кейс, когда тим лид просто-напросто заблокировал РАБОЧИЙ код, потому что ему не понравилась одна строчка в коде (то, как было написано, работало, но НЕ соответствовало его ожиданиям и той архитектуре, которую он требовал). И, наверное, не было бы проблемы, если бы тим лид не заблокировал ветку для мержа в ночь перед завершением спринта и демо..

🗯 И, пока я слушала доклад, вот о чем думала: вариант, конечно, простой - уйти от code review к парному программированию, НО что, если это тот вариант решения, когда мы не решаем проблему, а просто уходим от неё, а проблема-то, на мой взгляд, в коммуникации и командной этике и зрелости. И, уйдя от код ревью, мы будем видеть аналогичные проблемы, но в другие моменты процесса разработки.

Что думаете вы: комментарии при код ревью, общение, или парное программирование? Или что-то другое? 💬

01.06.2022

#практика #доклад #аджайл_дэйз

Привет! 🫂 🗣💭Предлагаю поговорить на тему необходимости code review в формате переписки и комментариев, а так же тех случаев, когда код ревьюит тим лид/тех лид | Сетка — социальная сеть от hh.ru Привет! 🫂 🗣💭Предлагаю поговорить на тему необходимости code review в формате переписки и комментариев, а так же тех случаев, когда код ревьюит тим лид/тех лид | Сетка — социальная сеть от hh.ru Привет! 🫂 🗣💭Предлагаю поговорить на тему необходимости code review в формате переписки и комментариев, а так же тех случаев, когда код ревьюит тим лид/тех лид | Сетка — социальная сеть от hh.ru Привет! 🫂 🗣💭Предлагаю поговорить на тему необходимости code review в формате переписки и комментариев, а так же тех случаев, когда код ревьюит тим лид/тех лид | Сетка — социальная сеть от hh.ru Привет! 🫂 🗣💭Предлагаю поговорить на тему необходимости code review в формате переписки и комментариев, а так же тех случаев, когда код ревьюит тим лид/тех лид | Сетка — социальная сеть от hh.ru Привет! 🫂 🗣💭Предлагаю поговорить на тему необходимости code review в формате переписки и комментариев, а так же тех случаев, когда код ревьюит тим лид/тех лид | Сетка — социальная сеть от hh.ru