Выбор ответа Junior Код-ревью и правила команды спрашивают часто обновлено
Перед отправкой на ревью
Вопрос
Ассистент сгенерировал фичу, тесты зелёные. Что сделать перед тем, как звать коллегу на ревью?
Попробуйте сами
Выберите вариант, и откроется разбор.
Показать ответ и разбор
Коротко
Правильный ответ: Самому прочитать дифф целиком и убедиться, что понимаешь его. Ревьюер не должен быть первым читателем сгенерированного кода.
Варианты ответа
- Сразу отправить: ревьюер для того и нужен, чтобы читать весь код
- Самому прочитать дифф целиком и убедиться, что понимаешь егоправильный ответ
- Попросить ассистента переписать код ещё раз, для надёжности
- Удалить комментарии ассистента, чтобы код выглядел своим
Разбор
Ревьюер не должен быть первым читателем сгенерированного кода. Генерация стоит секунды, а чтение стоит чужого времени. Повторная генерация не заменяет проверку, а удаление комментариев ничего не проверяет.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.