Реджект (Reject) в Code-review – это отклонение кода, означающее, что он содержит ошибки, не соответствует стандартам или требует доработки перед слиянием в основную ветку репозитория.
Когда программист написал код и отправил его на проверку, коллеги могут найти ошибки или предложить улучшения. Если код требует доработки, его «реджектят» – отклоняют и просят исправить.
Процесс Code-review включает проверку кода перед слиянием. Если код не проходит проверку, ревьюер оставляет комментарии с замечаниями, и код отправляется на доработку.
Причины реджекта кода:
а) Логические ошибки – код работает неправильно или содержит баги.
б) Нарушение кодстайла – код не соответствует стандартам проекта.
в) Неоптимальная реализация – код можно сделать проще и эффективнее.
г) Отсутствие тестов – новый код может сломать старый функционал.
д) Низкая читаемость – сложный, запутанный код без комментариев.
Как происходит реджект в Code-review:
а) Разработчик создаёт pull request / merge request.
б) Ревьюер проверяет код и оставляет замечания.
в) Если есть проблемы, ревьюер не ставит аппрув (approve), а делает реджект (reject).
г) Разработчик исправляет ошибки и отправляет код на повторную проверку.
а) В командах разработчиков для обеспечения качества кода.
б) В open-source проектах, где код-ревью обязателен.
в) В CI/CD-процессах, где перед развертыванием код должен пройти проверку.
а) Разработчик сделал новый функционал, но в коде обнаружились ошибки. Тимлид реджекнул pull request с просьбой исправить.
б) Код написан не по стандартам проекта (например, не отформатирован). Ревьюер реджекнул его, попросив привести код в порядок.
в) В коде есть уязвимости. Безопасник реджекнул изменения, чтобы избежать потенциальных проблем.