Лид ревьюировал технические решения или только код после написания?
Что проверяют
Как был устроен инженерный процесс в вашей прошлой команде и какую роль в нём играли вы. Интервьюеру важно понять, привыкли ли вы согласовывать подход до начала работы или узнаёте о проблемах с архитектурой уже в пул-реквесте. Следом обычно спрашивают: «А вы сами предлагали решения на ревью или только получали замечания?»
Как отвечать
Опишите реальный процесс, без приукрашивания. Каркас такой:
- что ревьюилось до кода: крупные задачи сопровождались дизайн-заметкой или ADR, где описаны схема таблиц и миграции Flyway или Liquibase, контракт API в OpenAPI, выбор между синхронным REST и сообщениями через Kafka, границы модулей Spring;
- кто смотрел: лид, архитектор или вся команда на техническом груминге. Назовите формат и порог: например, всё, что меняет схему БД или публичный API;
- что ревьюилось после: пул-реквесты в GitLab или GitHub, обязательный аппрув и автоматические проверки до человека: тесты, SonarQube, Checkstyle или SpotBugs;
- ваша роль: писали ли вы такие заметки сами, ревьюили ли чужие решения, что изменилось благодаря этому.
Нужен свой пример: задача, где обсуждение подхода заранее сэкономило переделку, или наоборот, где его не было и на ревью выяснилось, что выбран не тот путь. Цифры берите свои. Если их не мерили, лучше описать эффект словами, чем выдумать процент.
Если лид смотрел только готовый код, так и скажите. Сильный ответ дальше: к чему это приводило (крупные переделки после ревью), что вы делали сами (коротко согласовывали подход в чате до начала работы) и что хотели бы видеть в новой команде. Этот же вопрос полезно задать работодателю в конце собеседования.
Где спотыкаются
Рассказывают идеальную картину из учебника. Интервьюер уточнит детали, и расплывчатые слова про «регулярные встречи» быстро выдадут, что процесса не было.
Жалуются на прошлого лида. Даже если ревью было формальным, говорите о процессе и последствиях, а не о человеке.
Путают ревью решения с ревью стиля. Замечания про отступы и имена переменных закрывает линтер, а вопрос именно про то, проверял ли кто-то подход до написания кода.