Soft Skills Best Practices: ревью в пятницу
Или как не быть мудаком в глазах коллег
Представьте себе ситуацию. Пятница, вторая половина дня. Часть команды, работающая в восточном часовом поясе, уже заканчивает эту неделю, но как всегда остаются задачи, которые хотелось бы доделать. Хотя бы чтоб мысли о рабочих моментах не отвлекали от отдыха. Потому что если на выходных тоже крутить в голове рабочие задачи, то можно выгореть и потерять продуктивность. Никому же не нужно потерять продуктивность, верно?
Вы - лид команды или сеньор, который хочет помочь коллегам и провести код-ревью. Вообще, его нужно было провести немного раньше, но все были очень загружены и только сейчас, в пятницу после обеда, дошли руки посмотреть МРы. Вы открываете репозиторий проекта и начинаете...
Первый коллега. Так, это скилловый сотрудник, он всю неделю решал сложную и очень важную для бизнеса задачу. Код сложен, но есть тесты и, в целом, он знает что делает. Бегло смотрим изменения, ставим аппрувы, переходим к следующему этапу.
Второй коллега. Он давно на проекте, но до сих пор не очень понимает предметную область, а его кодстайл заставляет глаза истекать кровью. Вы смотрите код и боже... любой метод более 10 строк кода вызывает чувство страха. Просто потому что сочетание навыков коллеги, странного кода и непонимания что именно он хочет сделать, может сломать стенд для деплоя. Вы копируете код, запускаете среду разработки и начинаете вникать как всё работает, по пути оптимизируя решение. Спустя пару часов и пару десятков комментариев к МРу вы заканчиваете ревью. То же самое делают и другие коллеги, в том числе с вашим кодом.
В понедельник оказывается что код первого коллеги зааффектил другую часть системы, код первого, в целом, работает, но он потратил на его исправления все выходные, работоспособность снижена на 20% на ближайшую неделю.
Вопрос: где было принято ошибочное решение?
Кто сейчас подумал что в самом начале, когда вы вообще сели за ревью - отлично. Само по себе решение сесть за ревью в самом конце рабочей недели - хреновая затея по нескольким причинам.
- Общая усталость. Она есть вне зависимости от того, насколько вы продуктивны и насколько вы её замечаете. Ваше внимание будет расфокусировано и на решения начнёт влиять субъективный фактор (например, личное отношение к коллеге или его прошлый опыт) гораздо больше чем обычно. С точки зрения вашего подсознания, нужно побыстрее сделать работу и отдыхать. И чем продуктивнее вы работали всю неделю - тем сильнее выражен этот эффект.
В итоге вы можете пропустить то, на что стоило обратить внимание. - Результаты ревью. Допустим, у вас нет комментариев или они незначительны - тогда коллега мержит свой реквест и всё прекрасно. А если нет? Особенно если вы действительно разбираетесь в предметной области и обладаете высокими скиллами. В этом случае вы пишете комментарии, что нужно поправить, а ваш коллега вынужден или править их здесь и сейчас (использовав для этого вечер пятницы) или ходить и обдумывать решение последующие дни.
Даже если такое поведение не характерно для вас, это не значит что он так не сделает. - Влияние на психологический климат в команде. Поставьте себя на место того, кому вы пишете комментарии. Вы должны их написать и все это знают. Но взявшись за это в конце рабочей недели, вы вредите и себе, и коллеге и бизнесу. Даже когда ваши комментарии объективны и правильны - вы всё равно мудак в глазах коллеги. От одного раза nothing не will be, как говорится, но ревью - процесс регулярный. И рано или поздно может сложиться так, что коллеги просто не захотят продолжать путь с вами. И бизнесу придётся решать эту проблему, либо переместив вас в другое место, либо попрощавшись.
Разумеется, этот список можно продолжить, но даже этих трёх пунктов хватит чтоб понять - просто не стоит проводить ревью в пятницу. Для многих, ревью - это простая и не особо важная задача, на которую можно использовать остаток дня, но потенциальный вред от этого в разы превышает пользу.
Во многих проектах существует негласное правило - никакого деплоя в прод в пятницу, иначе выходные станут рабочими днями. Но ревью может дать совершенно такой же эффект. Вряд ли вы хотите именно этого, верно?
Помните о тех, с кем вы в одной лодке, и ваши коллеги никогда не скажут при увольнении "Всё прекрасно, просто хочу попробовать себя в гетеросексуальном коллективе":)
P.S. Понравилось? Поставь лайк и поделись с другими. Не понравилось? Обсудим в коментариях, пиши!