Код-ревью — игра в кошки-мышки. Это может быть помощь новичку, второе мнение, детальная проверка перед деплоем. В ревью часто не проверяют каждую запятую, и, в зависимости от цели, некоторые вещи можно пропустить.
Не оскорбляйте
Всё как в жизни: никаких «ты тупой», «кто тебя учил прогать», «с таким кодом только дворником работать». Ничего подобного быть не должно ни в коем случае.
Уточняйте непонятное
Хорошо, если разработчик способен объяснить, почему выбрал какое-то решение в коде. Поэтому не надо давать готовых ответов, чтобы человек скопипастил и забыл. Он так ничему не научится, а вы так и будете постоянно править за ним.
Можно спросить «почему ты принял именно такое решение?». Правильных решений множество. Может рассуждения разработчика будут логичны и останется только немного улучшить его идею.
А вот проталкивать своё идеальное решение ревьюера не надо. Если единственное обоснование в том, что вы так хотите или вам кажется это лучшим.
Не стесняйтесь себя
Например, я задаю много вопросов, подвожу к ответам, но не даю их, показываю, где искать информацию, рассказываю плюсы и минусы решений, люблю рассуждать вместе и несмешно шутить.
Мне не стрёмно, если решение другого окажется лучше моего или он сможет меня переубедить. Наоборот, это круто, так ребята и растут. Кому-то такой стиль не подходит. Помню, один мой наставник был таким. Меня тогда это бесило.
«Почему ты не просто не можешь дать мне готовый ответ?! Это потому что ты сам нихрена не знаешь?!», — думала я тогда. Сейчас я ему очень благодарна.
Задавайте вопросы
Он может не знать всего, может ошибаться, ваше ревью может быть у него десятым за сегодня. Поэтому что? Задаём вопросы, обращаем внимание на места, где есть сомнения.
Делайте выводы
Ревью учит формулировать мысли, проверять, что все друг друга поняли до начала работы, быть честной, не бояться выражать свои мысли, продуктивно обсуждать идеи.
А тех, чью работу проверяют, ревью учит задавать точные вопросы, искать информацию, аргументировать, конструктивно обсуждать, принимать критику (здоровую).
Важно, как вы чувствуете себя после ревью. Если у вас появилось понимание, куда копать, где ошибка и почему, как ещё можно посмотреть на проблему — это хорошо.
Не бойтесь задавать вопросы, если сомневаетесь или что-то не поняли. Если вы не согласны, аргументируйте. Адекватный ревьюер не пошлёт вас, ответит на вопросы, обсудит ваши сомнения и оценит, что вы смотрите шире заданных рамок.