engineering-playbook/docs/advanced/00-code-review.md

Pull Requests: Стандарты Code Review

Отправленный на сервер код (Push) никогда не попадает в главную ветку автоматически. Для слияния (Merge) вашей рабочей ветки с main необходимо создать Pull Request (PR) — официальный запрос на включение ваших изменений в основной продукт корпорации.

Жизненный цикл Pull Request

Процесс слияния подчиняется строгим регламентам качества:

1. Создание PR: На платформе (Gitea/GitHub) вы открываете запрос. Описание PR должно чётко отражать суть внесённых изменений, логику архитектурного решения и содержать ссылки на связанные задачи (Issues), например: Fixes #404.
2. Назначение ревьюеров: К проверке кода обязательно привлекаются старшие разработчики (Tech-Leads) или профильные DevOps-специалисты из вашей организации.
3. Аудит кода (Code Review): Ревьюеры построчно проверяют ваш код на соответствие архитектурным стандартам, уязвимостям безопасности и производительности. Если найдены недочёты, ревьюер оставляет комментарии к конкретным строкам.
4. Доработка (Refining): Автор локально вносит исправления, делает новые коммиты и отправляет их (git push) в ту же ветку. Pull Request обновится автоматически.
5. Утверждение (Approve): Только после полного устранения всех замечаний ревьюер нажимает системную кнопку одобрения. Без аппрува система аппаратно заблокирует слияние.
6. Слияние (Merge): Код официально становится частью главной ветки.

Этот стандартизированный процесс полностью исключает человеческий фактор и предотвращает случайное попадание некачественного, нестабильного или уязвимого кода в Production среду.