Как на текущей работе проходит Code Review

В нашей команде Code Review — обязательный этап перед мержем кода в основную ветку. Используем GitHub/GitLab Pull Requests. Основные принципы:

1. Два апрува — минимум два разработчика должны одобрить изменения
2. Автоматические проверки — линтеры, тесты, сборка
3. Скорость — стараемся ревьюить в течение дня
4. Конструктивная критика — комментируем с предложениями по улучшению

Пример комментария в PR:

// Вместо ручного маппинга лучше использовать деструктуризацию:
- const name = user.name;
- const age = user.age;
+ const { name, age } = user;

Особое внимание уделяем:

• Читаемости кода
• Соответствию гайдлайнам
• Потенциальным багам
• Производительности

После правок автор сам мержит PR, если нет блокирующих замечаний.