Как происходит ревью на последнем месте работы

Ревью кода у нас проводилось через GitHub/GitLab. Перед мержем в основную ветку создавался PR, который должен был получить минимум 2 апрува от коллег.

Основные правила:

• Проверяем code style и соответствие гайдлайнам
• Смотрим на читаемость и поддерживаемость кода
• Проверяем покрытие тестами (если это новый функционал)
• Уделяем внимание потенциальным багам и edge cases

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

// Вместо такого подхода:
users.filter(u => u.active).map(u => u.name)

// Лучше разделить операции для читаемости:
const activeUsers = users.filter(u => u.active);
const activeUserNames = activeUsers.map(u => u.name);

Ревью проходило в дружелюбной атмосфере, с акцентом на обучение и улучшение кода.