Надо ли добавлять джуна в ревьюеры?

Во многих командах такой вопрос даже не стоит. Понятно, что новичок
▪️ мало разбирается в предметной области
▪️ плохо ориентируется в коде проекта

А значит ничего не поймёт, ничего не скажет, и нет смысла его добавлять в ревьюеры.

Даже если вы сеньор, дать джуниору посмотреть пул-реквест - хорошая идея. И тому есть четыре причины:

1️⃣ Проверка читаемости

Применяете SOLID, следуете принципам clean code, но понять вас способен только разработчик с 10-летним опытом? Что-то тут не так🙂

Возможно вы пишете запутанную дичь в стиле 2000-х, пул-реквест занимает 50 файлов, вы вышли за пределы задачи и заодно провернули сложный рефакторинг. Это сильно усложняет понимание кода. Но коллеги-сеньоры привыкли к вашему стилю, поэтому не жалуются.

А вот если изменения понятны даже новичку, значит вы написали действительно читаемый и понятный код. Чтобы помочь начинающим ревьюерам, напишите саммари пул-реквеста. 3-5 предложений: в чём задача, главная проблема и суть решения.

2️⃣ Быстрее получаете фидбэк

Не только джуниоры, но и другие коллеги быстрее поймут контекст и суть изменений. Быстрее дадут фидбэк или поставят аппрув.

3️⃣ Многоуровневый фидбэк

Каждый сделает ваш код лучше:
🔸 Джуниор заметит опечатку в названии переменной
🔸 Мидл подскажет удобный метод из java 11
🔸 Сеньор укажет на непокрытый кейс
🔸 Тимлид заметит несоответствие корпоративным стандартам

4️⃣ Обмен знаниями

Материалы в разделе онбординга устаревают. Пул-реквесты показывают то, что происходит здесь и сейчас. Код-ревью быстро вводит нового сотрудника в курс дела:
🔹 Над чем вообще работает команда
🔹 Как писать код и как его оформлять
🔹 Какие тесты и в каком объёме писать
🔹 Как работать с БД и накатывать апдейты
🔹 Что обновлять в документации и CI

Чем быстрее новички вольются в проект, тем быстрее возьмут на себя ваши задачки.