Ревью кода: правила для обеих сторон

Ревью кода: правила для обеих сторон

Проверяющему. Разделяйте обязательное и вкусовое: «здесь теряются данные» и «я бы назвал иначе» — разные вещи, и помечать это стоит явно. Объясняйте причину, а не только требование. Не переписывайте чужой код целиком в комментариях: автор ничего не поймёт, просто скопирует.

Автору. Не спорьте сразу — сначала переспросите, возможно, вы поняли замечание не так. Не приносите на ревью двадцать файлов: большие изменения смотрят формальнее и пропускают чаще. И благодарите за найденную ошибку — её нашли до пользователей.

Общее правило: ревью не про то, кто умнее, а про то, чтобы через год этот код мог менять кто-то третий.

Если ревью в вашей команде нет вовсе, просите разовые разборы у опытного коллеги — иначе рост идёт вслепую.

IT Иваново — t.me/itnews_ivanovo