Сколько фронтендеров участвуют в code review
Что проверяют
Как у вас устроено ревью и какая у вас в нём роль. Ждут не число, а процесс: сколько одобрений нужно на мердж, кто их даёт, что смотрит человек, а что уже проверила автоматика до него.
Как отвечать
Стройте ответ вокруг правил, а не вокруг количества людей:
- сколько одобрений требует мердж и есть ли обязательный владелец кода через
CODEOWNERS; - как назначаются ревьюеры: автоматически по зоне кода или автор выбирает сам;
- какой размер PR считается нормальным и что делают с гигантскими;
- сколько ждут ревью: если больше дня, работа встаёт, и обычно появляется договорённость о времени реакции;
- что отдано линтеру, форматтеру, типам и тестам в CI, чтобы на ревью не спорили о кавычках и порядке импортов;
- кто смотрит доступность, поведение на разных разрешениях и размер бандла: на фронте этим часто не занимается никто, и если у вас занимались, скажите об этом отдельно.
Если вы задаёте этот вопрос работодателю, добавьте про тон: можно ли блокировать мердж из-за стиля и чем заканчиваются споры, когда ревьюер и автор не сходятся во мнении.
Где спотыкаются
Сводят ответ к числу вроде «смотрят двое». Интервьюеру нужно понять, ревью у вас инженерная практика или формальная кнопка перед мерджем, а из числа это никак не следует.
Второе: описывают идеальный процесс, которого не было. Достаточно спросить, что делали с изменением на две тысячи строк перед релизом, и красивая картинка перестаёт сходиться.
Третье: не отделяют ревью от статического анализа. Если люди вручную вылавливают неиспользуемые импорты и съехавшее форматирование, это не строгость команды, а не настроенный CI, и говорить об этом как о достижении не стоит.