Ревью кода: правила для обеих сторон
Проверяющему: разделяйте обязательное и вкусовое — «здесь теряются данные» и «я бы назвал иначе» разные вещи. Объясняйте причину, а не только требование. Не переписывайте чужой код целиком в комментариях.
Автору: не спорьте сразу, сначала переспросите. Не приносите двадцать файлов одним запросом: большие изменения смотрят формальнее. И благодарите за найденную ошибку — её нашли до пользователей.
Для распределённых команд добавляется своё: ревью через переписку требует более полных формулировок. «Почему так?» без пояснения читается как претензия, когда собеседник не видит вашего лица и ответит только завтра.
Пишите развёрнуто — это дешевле, чем сутки на выяснение тона.
IT Улан-Удэ — t.me/it_ulan_ude
IT Улан-Удэ
@it_ulan_ude
Ревью кода: правила для обеих сторон
Этот пост опубликован в Telegram-канале IT Улан-Удэ. Подписаться можно по ссылке: @it_ulan_ude.