На что обращаешь внимание в первую очередь при проведении code review

При code review в первую очередь смотрю на:

1. Читаемость кода - понятные названия переменных/функций, соблюдение PEP8, отсутствие "магических чисел"
2. Архитектуру - разделение ответственностей, отсутствие дублирования, правильное использование паттернов
3. Безопасность - санитизация ввода, защита от инъекций, корректная обработка ошибок
4. Производительность - оптимальные алгоритмы, отсутствие лишних операций в циклах
5. Тестируемость - модульность кода, отсутствие жестких зависимостей

Пример плохого кода:

def p(d):  # непонятное название
    for i in d:  # магическое число 5
        if i > 5:
            print(i)

Пример улучшенного:

def print_numbers_greater_than_threshold(numbers, threshold=5):
    """Выводит числа превышающие пороговое значение"""
    for number in numbers:
        if number > threshold:
            print(number)

Также проверяю соответствие задаче и наличие документации для сложных участков.