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