- **Читаемость кода**: Проверяю отступы, названия переменных/функций (PEP 8), докстринги. - **Логика изменений**: Соответствует ли задача в тикете коду, нет ли скрытых багов. - **Тесты**: Добавлены ли новые тесты, проходят ли существующие. - **Конфликты**: Есть ли merge-конфликты с основной веткой. - **Производительность**: Нет ли явных проблем (напр., O(n²) там, где можно O(n)). - **Безопасность**: Нет ли уязвимостей (SQL-инъекции, hardcoded credentials). - **Документация**: Обновлены ли 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 ```