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