Два сильных разраба воевали в ревью. Разделил им зоны
У меня в команде есть два сильных разработчика. Оба с опытом, оба умные, и оба упертые до невозможности.
Сначала они помогали друг другу и спорили по делу. Но со временем их ревью превратились в бесконечные перепалки. Один постоянно правил стиль кода, другой откатывал эти правки обратно. У одного «отступ не по гайду», у второго — «оптимизация ради оптимизации». Пулл-реквесты начали висеть по неделе — каждый ждал, когда другой первым сдастся.
Других товарищей это тоже тормозило. Никто не хотел ревьюить их код, потому что обсуждения превращались в философские баталии на двадцать комментариев.
Поэтому я позвал обоих на разговор и попросил объяснить, в чем суть конфликта. Оказалось, оба считают себя ответственными за качество, но каждый понимает его по-своему. Один за чистоту и структуру, второй — за скорость и практичность.
Мы договорились на самое простое — разделить зоны. Один пусть ревьюит архитектуру и отвечает за общие подходы. А второй — за конкретную реализацию и тесты. Если же вопрос спорный, то все решаем втроем. На коротком созвоне, а не в комментариях.
Месяц работаем по такой схеме, пока все спокойно. Иногда они все еще цепляются, но теперь хотя бы слушают друг друга. Посмотрим, надолго ли их хватит.