Ответ вслух Middle Код-ревью обновлено

Автор не согласен с замечанием

Вопрос

Как ты поступишь, если автор PR не согласен с твоим замечанием?

Попробуйте сами

Ответьте вслух за 2 минуты, как на собесе. Потом сверьтесь с тезисами: что прозвучало, а что нет.

Показать ответ и разбор

Коротко

Сначала спрашиваю, почему он сделал именно так: часто у автора есть контекст, которого у меня нет. Потом объясняю свою позицию через последствия: не «так некрасиво», а «при повторном клике отправятся два платежа».

Что должно прозвучать в ответе

  • Уточнить его аргументы, возможно, я чего-то не знаю

  • Объяснить свою позицию через последствия, а не вкус

  • Отличать принципиальное (баг, безопасность) от вкусового; вкусовое уступить

  • Если спор затянулся, обсудить голосом

  • Если нет согласия, позвать третьего или опереться на договорённости команды и зафиксировать их

Что могут спросить следом

  • Был ли у тебя такой случай?
  • Когда ты блокируешь мерж?

Разбор

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

Ответьте сами, а не прочитайте

В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.