Как проходило Code Review на прошлом проекте

На прошлом проекте Code Review было обязательным этапом перед мержем в основную ветку. Использовали GitHub Pull Requests с чек-листом:

1. Автоматические проверки:
   • flake8 для стиля кода
   • pytest для тестов
   • mypy для типизации
   • SonarQube для анализа качества

2. Ручной ревью:
   • Минимум 2 апрува от команды
   • Проверяли:
     • Соответствие архитектуре
     • Качество тестов
     • Читаемость кода
     • Оптимальность решений

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

# Вместо этого:
results = [x for x in data if x > 0]

# Лучше так - более явно:
positive_numbers = [num for num in data if num > 0]

Особое внимание уделяли балансу между perfectionism и delivery - не затягивали ревью, но и не пропускали явные проблемы.