코드 리뷰와 병합 방식
6. 연습 문제
문제 1. 리뷰 상태 구분
리뷰어가 "Request changes" 를 남긴 PR 을 병합하려면 먼저 무엇이 필요한가요?
같은 리뷰어의 재승인(Approve)이 필요합니다. Request changes 는 재승인
전까지 병합을 막는 상태이므로, 지적을 반영한 커밋을 올리고 재검토를 요청합니다.문제 2. CODEOWNERS 우선순위
다음 CODEOWNERS 에서 docs/guide.md 의 소유자는 누구인가요?
* @team-core
docs/* @doc-team
docs/guide.md @writer-lee
# 답: @writer-lee (가장 아래, 가장 구체적인 규칙이 우선)문제 3. 병합 방식 선택
커밋 10개가 섞인 실험적인 브랜치를 깔끔한 한 줄로 합치고 싶습니다. 어떤 병합 방식을 쓰나요?
# 답: squash 병합
git merge --squash feature/experiment
git commit -m "feat: 실험 기능 정리 (#12, squash merge)"문제 4. force push 시나리오
브랜치 보호가 걸린 main 에 실수로 올라간 커밋을 고치려 합니다. 올바른 방법은 무엇인가요?
# 답: 되돌리지 않고 새 커밋을 쌓는다 (reset --hard 뒤 force push 는 거절됨)
git add .
git commit -m "fix: 실수 수정"
git push origin main문제 5. rebase 후 push
이미 GitHub 에 올린 feature 브랜치를 git rebase main 한 뒤 다시 올리는 명령을 쓰세요.
# 답: 일반 push 는 non-fast-forward 로 거절되므로 아래가 필요하다
git push --force-with-lease origin feature/mybranch