1. Код-ревью – это не подвернувшаяся возможность поглумиться над коллегой. Некоторые коллеги с комплексом неполноценности упражняются в остроумии в комментариях к пулл-реквесту. Это характеризует ревьюера с плохой стороны, и сильно демотивирует автора кода. А если код очень "пахнет"? Да, такое может быть, хотя удивительно, как пропустили на техсобесе очевидного бракодела; тогда это выносится на 1-1 у тимлида.

2. Не описывайте каждое нарушение стиля кода, если они однородные. Это даже не ИИ умеет, это обычный линтер умеет. Напишите в низу пулл-реквеста: “запусти линтер и исправь замечания.”

3. Если код-ревьюемый ПР подразумевает исправление вашего кода, будьте объективными и готовыми к тому, что кто-то вас исправляет. Вы не поверите, насколько нервозными могут становиться разработчики, когда кто-то вносит правки в их “идеальный” код.

4. Самые горькие ошибки, которые призвано исправить код-ревью, часто всплывают там, где изменений не было. Не поленитесь открыть код у себя в IDE и прощелкать по зависимостям изменённого кода.

Спасибо за внимание