На что обратишь внимание при получении pull request

1. Читаемость кода: Проверяю отступы, названия переменных/функций (PEP 8), докстринги.
2. Логика изменений: Соответствует ли задача в тикете коду, нет ли скрытых багов.
3. Тесты: Добавлены ли новые тесты, проходят ли существующие.
4. Конфликты: Есть ли merge-конфликты с основной веткой.
5. Производительность: Нет ли явных проблем (напр., O(n²) там, где можно O(n)).
6. Безопасность: Нет ли уязвимостей (SQL-инъекции, hardcoded credentials).
7. Документация: Обновлены ли README/comments при изменении API/логики.

Пример ревью-комментария:

# Было
def calc(a, b): return a * b

# Стало (лучше)
def calculate_product(x: int, y: int) -> int:
    """Return the product of two integers."""
    return x * y