Документация

Рецензирование и объединение результата

Что поступает в pull request, что проверять и как отправлять работу на доработку.

Coroid завершает процесс созданием pull request в отдельной ветке, основанной на вашей базовой ветке. Слияние не происходит автоматически — это решение остаётся за вами.

Что с этим приходит

  • Изменения кода
  • Тесты, написанные или обновлённые вместе с кодом
  • Результаты выполнения вашего существующего набора тестов
  • Результаты прохождения любых правил контроля качества, настроенных в вашей организации
  • Проверка различий в коде с уже исправленными или указанными замечаниями

Суть не в замене вашей проверки — ваша проверка должна быть не первой.

На что именно обратить внимание

Механические аспекты уже проверены. Сосредоточьтесь на том, что может оценить только вы:

  1. Решает ли это нужную проблему? Сопоставьте результат с критериями приемки из технического задания, а не с вашим воспоминанием о запросе.
  2. Соответствует ли это структуре кода? Соблюдение стандартов, именования, структуры абстракций. Coroid анализирует ваш код, чтобы определить эти параметры, и почти всегда делает это верно — но не всегда.
  3. Какие части кода затронуты неожиданно? Посмотрите на список файлов перед просмотром различий.
  4. Являются ли новые тесты полезными? Тест, который ничего не проверяет, хуже, чем его отсутствие.

Отправка работы на доработку

У вас три варианта, от наименее трудоёмких к более сложным:

  • Оставить комментарий и запросить доработку — задача возвращается агенту-разработчику с вашими замечаниями, сохраняя контекст. Подходит для случаев «всё верно, но неполно».
  • Отклонить план и запустить повторно — когда подход к решению ошибочен, а не реализация.
  • Отменить задачу — когда работа вообще не нужна. Отмена останавливает выполнение и освобождает слот агента.

Будьте конкретны в комментариях по доработке, как с коллегой. «Здесь не обрабатывается пустой случай в parseRange" приводит к исправлению; «нуждается в доработке» — к догадкам.

Слияние

Слияние осуществляется через вашего провайдера с использованием стандартных правил защиты веток, обязательных проверок и правил утверждения. Coroid создаёт pull request; ваш существующий процесс определяет дальнейшие действия с ним.

В слиянии нет ничего особенного — это намеренно. pull request — обычная ветка, поэтому она проходит через ваши существующие механизмы проверки и CI, которым вы доверяете.

Далее

Куда перейти дальше — ссылки ведут в другие разделы документации в зависимости от вашей задачи.