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

На прошлом месте код-ревью проводилось через GitHub/GitLab. Перед мерджем в основную ветку создавался пул-реквест, который проверяли минимум 2 разработчика. Основные критерии:

• Читаемость: соблюдение PEP 8, понятные названия переменных/функций.
• Производительность: избегание лишних циклов, оптимизация запросов.
• Тестируемость: покрытие тестами новых фич.
• Безопасность: санитизация ввода, проверка прав доступа.

Пример замечания:

# Было
def process_data(data):
    res = []
    for i in data:
        if i % 2 == 0:
            res.append(i * 2)
    return res

# Стало (list comprehension)
def process_data(data):
    return [x * 2 for x in data if x % 2 == 0]

Ревьюеры оставляли комментарии прямо в PR, иногда с примерами исправлений. Конфликты разрешались в чате или созвоне. После правок — повторная проверка.