Обратная связь в айтишке: запрет на токсичность в код-ревью

Раньше у нас такого процесса не было: синьоры были заняты своими задачами, а джуны росли только благодаря своим усилиям. Обратной связи просто не было.

А на другом проекте старшие разработчики самоутверждались за счет младших. В комментариях сквозило "что за ерунду ты написал?" и "ты это серьёзно". В итоге, джуны боялись показывать свой код, а кто поактивнее, тот ругался, а со временем уходил из компании.

В моей команде действует простое правило: не критиковать человека.

Мы обсуждаем только код.

Ведь код-ревью может убить команду, а может сделать ее еще сильнее. Вместо "это ужасно, переделай" мы пишем "а что если попробовать вот такой подход?" или "это приведёт к таким-то ошибкам, лучше сделать иначе". И вместо "ты ошибся" — "я не до конца понял эту логику, можешь пояснить?"

Звучит просто? Ага. Но результат есть: минус токсичность — плюс команде. И мои красавчики это подтвердят.

Код-ревью превратилось из экзамена в совместную работу. В сеанс обмена знаниями. Сеньоры/лид помогают и учат, а не унижают. Джуны перестали бояться и начали задавать вопросы.

Становимся сильнее как команда, а код стал — все лучше и лучше. Главное же: мы научились уважению друг к другу.

Артель Артемия

В этом посте были ссылки, но мы их удалили по правилам Сетки