Как было устроено код-ревью на предыдущей работе

На предыдущем месте код-ревью проводилось через GitHub/GitLab. Каждый пулл-реквест (PR) должен был:

1. Иметь описание изменений
2. Включать тесты (если это новая функциональность)
3. Пройти CI/CD пайплайн
4. Получить минимум 2 апрува от коллег

Ревьюеры проверяли:

• Соответствие код-стайлу проекта (flake8/black)
• Корректность архитектурных решений
• Наличие edge-case обработок
• Читаемость кода

Пример процесса:

# Ветка feature/new-endpoint
def process_data(data: list) -> dict:
    if not isinstance(data, list):  # Добавили проверку типа
        raise ValueError("Expected list")
    return {item['id']: item for item in data}

Комментарии в ревью выглядели так:
⚠️ Нет обработки пустого списка → добавляли if not data: return {}

Особенности:

• Время ревью не должно было превышать 24 часа
• Критику всегда сопровождали предложением решения
• Автоматические линтеры сокращали 30% ручной работы