문서

결과물을 리뷰하고 병합하기

pull request에 도착하는 내용, 확인할 사항 및 작업을 반려하는 방법

Coroid는 자체 브랜치에서 pull request를 생성하여 기본 브랜치와 비교한 뒤 작업을 완료합니다. 이때 병합은 이루어지지 않습니다. 최종 결정권은 사용자에게 있습니다.

함께 제공되는 내용

  • 코드 변경 사항
  • 해당 코드와 함께 작성되거나 업데이트된 테스트
  • 기존 테스트 스위트를 실행한 결과
  • 조직에서 설정한 모든 품질 검사의 결과
  • 변경된 부분에 대한 리뷰 결과로, 이미 수정된 사항이나 목록으로 정리된 문제점이 포함됩니다.

목적은 리뷰 과정을 대체하는 것이 아닙니다. 단지 사용자의 리뷰가 첫 번째 단계가 되지 않도록 하는 것입니다.

실제로 확인해야 할 사항

기계적인 검사는 이미 완료되었습니다. 사용자가 판단해야 하는 사항에 집중하세요:

  1. **올바른 문제를 해결하고 있는가?**요구사항에 명시된 수용 기준과 비교해 보세요. 자신의 기억 속 요청 내용과 비교하지 마세요.
  2. **코드베이스에 잘 맞는가?**명명 규칙, 추상화 구조 등을 확인하세요. Coroid는 코드를 분석하여 이를 파악하지만, 항상 정확한 것은 아닙니다.
  3. **예상치 못한 부분이 변경되었는가?**변경된 부분을 확인하기 전에 파일 목록을 살펴보세요.
  4. **새로 추가된 테스트가 유의미한가?**아무것도 보장하지 않는 통과된 테스트는 테스트가 없는 것보다 더 나쁩니다.

작업을 반려하는 방법

사용할 수 있는 옵션은 세 가지이며, 처리 비용은 점차 증가합니다:

  • 코멘트를 남기고 재작업을 요청하기— 작업 내용과 함께 사용자의 의견이 개발자 에이전트에게 전달되며, 기존 컨텍스트는 그대로 유지됩니다. '올바르지만 미완성'인 경우에 적합합니다.
  • 계획을 거부하고 다시 실행하기— 접근 방식에 문제가 있을 때 사용하며, 실행 과정에는 문제가 없음을 의미합니다.
  • 작업 취소하기— 해당 작업을 전혀 진행하지 않아야 할 때 사용합니다. 취소하면 실행이 중단되고 에이전트 슬롯이 확보됩니다.

재작업 관련 코멘트는 동료에게 전달하듯 구체적으로 작성하세요. "이 코드는 빈 값 상황을 처리하지 못합니다"parseRange"로 수정안을 제시하고, "개선 필요"라는 표현은 추측에 불과합니다.

병합하기

기존 제공업체를 통해 병합을 진행하며, 일반적인 브랜치 보호 규칙, 필수 검사 및 승인 절차를 적용합니다. Coroid는 pull request를 생성하지만, 이후 처리는 사용자의 기존 프로세스에 따릅니다.

병합 과정에 특별한 점은 없습니다. 이는 의도적인 설정입니다. pull request는 일반적인 요청이므로 기존에 신뢰하는 리뷰 및 CI 프로세스를 그대로 따릅니다.

다음 단계

다음으로 이동할 곳— 사용자의 목적에 따라 문서의 다른 섹션으로 이동할 수 있습니다.