Выбор ответа Middle Код-ревью обновлено
Огромный пулреквест
Вопрос
Тебе прислали PR на 3000 строк, где рефакторинг смешан с новой фичей. Как лучше поступить?
Попробуйте сами
Выберите вариант, и откроется разбор.
Показать ответ и разбор
Коротко
Правильный ответ: Попросить разделить: рефакторинг отдельно, фича отдельно. Большие смешанные PR плохо ревьюятся: внимание падает, баги проскакивают.
Варианты ответа
- Попросить разделить: рефакторинг отдельно, фича отдельноправильный ответ
- Апрувнуть, раз тесты зелёные, а вопросы обсудить после релиза
- Переписать всё самому, так быстрее, чем объяснять автору
- Оставить 200 комментариев-нитпиков, чтобы автор всё поправил
Разбор
Большие смешанные PR плохо ревьюятся: внимание падает, баги проскакивают. Разделение позволяет быстро принять рефакторинг (поведение не меняется) и внимательно проверить фичу. Слепой апрув переносит риск в прод. Переписывать за автора неуважительно и не масштабируется. Поток мелочей тонет и не помогает главному.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.