14강. 코드 리뷰와 PR 병합 흐름

?1. 이번 강의에서 해결할 문제

리뷰를 개인 평가로 받아들이거나 GitHub에서 파일을 임의 수정하면 로컬과 원격 흐름이 어긋납니다. 근거가 남는 대화와 커밋으로 피드백을 반영합니다.

✓2. 학습 목표

✓이 강의를 마치면

• 리뷰의 Comment, Approve, Request changes를 구분합니다.
• 리뷰 수정을 같은 기능 브랜치에 추가합니다.
• merge, squash, rebase 병합의 이력 차이를 선택합니다.

◇3. 핵심 개념

리뷰는 사람의 판단과 자동 검사 결과를 결합합니다. 수정 요청은 새 PR이 아니라 같은 브랜치의 후속 커밋으로 반영합니다. GitHub는 저장소 설정에 따라 merge commit, squash merge, rebase merge를 제공하며 팀의 이력 정책을 따릅니다.

§4. 단계별 명령어와 예시

브랜치: feature/profile-page

PowerShell

Set-Location C:\dev\git-learning-site
git switch feature/profile-page
git pull --ff-only
Add-Content profile.md "`n## 다음 학습 목표"
git diff --check
git add profile.md
git commit -m "docs: clarify next learning goal"
git push

GitHub PR의 Files changed에서 수정 내용을 다시 보고, 대화에 무엇을 바꿨는지 답합니다. 필수 검사가 통과하고 리뷰가 끝난 뒤 저장소 정책에 맞는 병합 버튼을 선택합니다.

PowerShell · 병합 후 정리

git switch main
git pull --ff-only
git branch -d feature/profile-page
git fetch --prune

→5. 명령어가 동작하는 이유

같은 원격 브랜치에 push하면 열린 PR의 비교 범위가 자동 갱신됩니다. 병합 뒤 로컬 main을 pull해야 GitHub에서 생성된 병합 결과를 받습니다. fetch --prune은 원격에서 삭제된 브랜치의 오래된 추적 참조를 정리합니다.

!6. 자주 하는 실수와 안전한 해결법

!자주 하는 실수

• 리뷰마다 새 PR 생성: 기존 PR 브랜치에 후속 커밋을 push합니다.
• 검사 실패를 무시하고 병합: 로그에서 첫 실제 오류를 해결하고 다시 실행합니다.
• 병합 직후 로컬 브랜치를 강제 삭제: main을 갱신한 뒤 -d로 안전하게 삭제합니다.

✎7. 직접 실습

✎직접 실습

1. 본인 PR에 개선 의견을 하나 남깁니다.
2. 로컬에서 수정 커밋을 push해 PR이 갱신되는지 봅니다.
3. 검사 결과와 Files changed를 확인한 뒤 병합하고 로컬 main을 동기화합니다.

?8. 이해 점검 질문 3개

?이해 점검

1. 리뷰 반영을 같은 PR에 추가하려면 어느 브랜치에 push해야 하나요?
2. squash merge는 여러 기능 커밋을 main에 어떻게 남기나요?
3. 원격에서 삭제된 브랜치 참조를 정리하는 fetch 옵션은 무엇인가요?

■9. 핵심 요약

■핵심 요약

리뷰는 요구사항과 검증을 명확하게 만드는 협업 기록입니다. 같은 브랜치에서 수정하고, 검사와 승인을 확인해 정책에 맞게 병합한 뒤 로컬을 정리합니다.