Как было организовано code review на прошлой работе

На прошлом проекте использовали GitHub Pull Requests для code review. Каждый разработчик создавал feature-ветку, после завершения работы — PR в main.

Процесс:

1. Автоматические проверки: CI запускал сборку, тесты и статический анализ (SonarQube).
2. Ревью командой: 2+ участника проверяли код (часто асинхронно). Акцент на:
   • Читаемость и согласованность с кодстайлом.
   • Отсутствие "магических чисел", дублирования.
   • Корректность архитектурных решений.
3. Комментарии: Прямо в GitHub, с предложениями правок. Пример:
   java

   Копировать

   // Было: 
   if (list.size() > 0) { ... }
   // Ревьювер: "Используй !isEmpty() для ясности"

4. Мерж: После исправлений и апрува PR вливался через squash-commit.

Нюансы:

• Для срочных фиксов допускалось ускоренное ревью (1 апрув + тесты).
• Конфликты стилей решались через pre-commit hooks (Checkstyle).