KUKJIN LEE
Posted 3 months ago
Approved and Merged, 코드 승인과 병합에 대해서
Approved란 무엇인가요?
Approved(승인됨)은 Pull Request(PR)가 코드 리뷰어들에 의해 검토된 후, 병합해도 좋다는 동의를 받은 상태를 의미합니다. PR을 생성한 후, 팀원들이 코드를 검토하고, 모든 리뷰어가 해당 PR의 변경 사항에 동의하면, PR은 "Approved" 상태가 됩니다. (즉 팀장 이상 또는 권한이 주어진 사람이 담당합니다.)
Approved 상태의 주요 특징
-
코드 검증: Approved는 변경된 코드가 팀의 코드 품질 기준을 충족하며, 병합해도 안전하다는 것을 의미합니다.
-
피드백 반영: 코드 리뷰 과정에서 제기된 피드백이 적절히 반영되었음을 나타냅니다.
-
협업 과정 완료: 팀원 간의 협업이 성공적으로 이루어졌음을 의미하며, 다음 단계인 병합(Merge)으로 넘어갈 준비가 되었음을 나타냅니다.
Merged란 무엇인가요?
Approved 상태의 Pull Request가 기본 브랜치(main 또는 master)로 병합되어, 해당 브랜치의 코드베이스에 포함된 상태를 의미합니다. 병합 과정에서는 Pull Request에서 제안된 변경 사항이 실제 코드베이스에 반영됩니다.
Merged 상태의 주요 특징
-
코드 반영: PR에서 변경된 코드가 실제 코드베이스에 통합되어 프로젝트의 일부가 됩니다.
-
기능 배포 준비 완료: Merged 상태의 코드는 배포(deployment)나 릴리스(release) 과정에서 사용될 준비가 되어 있습니다.
-
브랜치 정리 가능: 병합 후에는 PR을 생성했던 브랜치를 삭제하여 프로젝트를 깔끔하게 유지할 수 있습니다.
Approved와 Merged의 과정
-
Pull Request 생성
-
새로운 기능이나 버그 수정 내용을 포함하는 브랜치를 기반으로 Pull Request를 생성합니다.
-
-
코드 리뷰
-
팀원들이 PR을 리뷰하고, 필요 시 피드백을 제공합니다. 리뷰어가 코드에 만족하면 PR을 승인(Approve)합니다.
-
-
수정 및 재검토 (선택 사항)
-
피드백이 제공되면, PR 작성자는 이를 반영하여 코드를 수정하고, 다시 리뷰를 요청할 수 있습니다.
-
-
PR 승인 (Approved)
-
모든 리뷰어가 PR을 승인하면, PR은 Approved 상태가 됩니다.
-
-
PR 병합 (Merged)
-
PR이 승인되면, 해당 브랜치를 기본 브랜치에 병합할 수 있습니다. 병합이 완료되면 PR은 Merged 상태가 됩니다.
-
-
브랜치 삭제 (선택 사항)
-
병합된 브랜치는 더 이상 필요하지 않으므로, 삭제하여 저장소를 정리할 수 있습니다.
-
마무리
Approved와 Merged는 GitHub에서 Pull Request 작업이 완료되는 중요한 단계들입니다. 이 과정들을 통해 코드가 검토되고, 프로젝트에 안전하게 통합되며, 개발팀은 안정적인 코드베이스를 유지할 수 있습니다.