❓ Как отревьюить большой PR?
Большой PR — это архитектурная проблема, но раз он уже есть, вот стратегия:
🔹 Начинать не с первого файла в списке, а с точки входа — контроллера, команды или сервиса, где меняется логика.
🔹 До открытия первого файла — ответить на три вопроса: что должно измениться в поведении системы, что не должно, и где граница между старым и новым.
🔹 Тесты читать первыми, т.к. они документируют намерение автора лучше, чем любой комментарий.
🔹 Искать не ошибки, а допущения. Где код предполагает, что данные всегда валидны? Что очередь не упадёт? Что транзакция атомарна? Именно здесь прячутся баги, которые выстреливают через месяц.
🔹 Замечания по стилю без линтера — не тема для PR-комментариев. Если в проекте нет phpcs или php-cs-fixer, лучше их настроить, чем воевать в ревью.
Post #1254
1.23K
- ❤ 7
- 🔥 5
- 👍 2