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

На прошлой работе код-ревью проводилось через Pull Requests (PR) в GitHub. Процесс выглядел так:

1. Разработчик создает PR, назначает ревьюверов (обычно 1-2 коллеги из команды)
2. Ревьюверы проверяют код на:
   • Соответствие код-стайлу (через анализаторы вроде SonarQube)
   • Наличие Unit-тестов для новой логики
   • Оптимальность алгоритмов
   • Возможные edge-cases
3. Комментарии оставляются прямо в PR, обсуждаются в треде
4. После исправлений PR апрувится и мержится

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

// Вместо этого:
foreach(var item in list) { ... }

// Лучше использовать LINQ:
list.ForEach(item => ... );

Важные нюансы:

• Время ответа на PR не должно превышать 1 рабочего дня
• Критика только по коду, без перехода на личности
• Обязательно отмечать и хорошие решения тоже