Выбор ответа Middle Код-ревью и правила команды спрашивают часто обновлено
Пулреквест на 1500 строк
Вопрос
Коллега с помощью агента прислал пулреквест на 1500 строк: фича, рефакторинг и переформатирование вместе. Как поступить?
Попробуйте сами
Выберите вариант, и откроется разбор.
Показать ответ и разбор
Коротко
Правильный ответ: Попросить разбить: фича, рефакторинг и механика отдельно. Огромный дифф невозможно качественно отревьюить, его одобряют не читая.
Варианты ответа
- Одобрить: тесты зелёные, а читать столько нереально
- Попросить ИИ-ревьюер проверить и одобрить по его итоговому отчёту
- Пролистать и оставить пару замечаний по стилю для вида
- Попросить разбить: фича, рефакторинг и механика отдельноправильный ответ
Разбор
Огромный дифф невозможно качественно отревьюить, его одобряют не читая. Разбиение на логические части делает ревью реальным. ИИ-ревьюер полезен как первый фильтр, но решение остаётся за человеком.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.
Ещё по теме «Код-ревью и правила команды»
- Плюсы и минусы ИИ-ревьюера
- Правила автора пулреквеста с ИИ
- Свой хелпер вместо существующего
- Пометка «сгенерировано ИИ»
- Новая зависимость от ассистента
- Код-ревью: эндпоинт из пулреквеста, написанный с ИИ
- Файл правил проекта для ассистента
- Перед отправкой на ревью