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