Обязательное ревью кода — де-факто стандарт индустрии по улучшению качества кода. Но так было не всегда. Во всяком случае в нашей команде.
10+ лет назад у нас не было ревью кода. Сообщения о коммитах в svn приходили письмами на рассылку. Ее читали все желающие. Можно было ответить письмом на коммит — и это было “ревью”.
В качестве первого инструмента для ревью кода был внедрен Crucible из-за его нативной интеграции с Jira — продуктом той же Atlassian. Тогда еще у нас не было своего трекера.
Отныне коммиты в trunk стали возможными только после прохождения ревью кода.
Последующие несколько недель коллеги ходили злыми, как черти 😈
Разработчики воспринимали критику своего кода как личное оскорбление. Нужно время и опыт, чтоб научиться разделять критику кода от самого себя. Это, правда, непросто.
Мой коллега вернулся из отпуска и создал свое первое ревью кода:
- один файл
- 2 ревьювера
- 100+ комментов
Вечером мы с коллегой пили чай, и я решил донести до него свою точку зрения. В ответ услышал примерно следующее: — Саша, больше не слова, а то я тебя ударю…
Если учесть вспыльчивый характер моего коллеги — то были не пустые угрозы. Дабы избежать рукоприкладства, я решил дать время коллеге остыть и поговорить с ним на следующей день :)