Код-ревью
Вопрос
На что ты смотришь на код-ревью и как даёшь комментарии?
Попробуйте сами
Ответьте вслух за 2 минуты, как на собесе. Потом сверьтесь с тезисами: что прозвучало, а что нет.
Показать ответ и разбор
Коротко
Что должно прозвучать в ответе
сначала понять задачу и границы изменения
корректность и краевые случаи важнее стиля
стиль и форматирование должны проверяться линтером, а не человеком
комментарий формулируется как вопрос или предложение, а не приговор
различать блокирующее замечание и пожелание
хвалить удачные решения - ревью не только про недостатки
Что могут спросить следом
- Что делать, если автор не согласен
- Как ревьюить большой PR
- Где граница между замечанием и вкусовщиной
Разбор
Сначала читаю описание задачи: без него ревью превращается в вычитку синтаксиса. Дальше по приоритету: корректность и краевые случаи, безопасность и данные, границы модулей, читаемость, тесты. Стиль не обсуждаю - это работа prettier и eslint, спор о нём на ревью всегда потеря времени.
Форма комментария важна не меньше содержания: «а что будет, если массив пустой?» вместо «тут баг». Отделяю блокирующее от необязательного явной пометкой - иначе автор либо переделывает всё, либо игнорирует всё.
Отдельно: если решение хорошее, я это пишу. Ревью, в котором только замечания, люди начинают воспринимать как нападение, и оно перестаёт работать.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.