PR 코드리뷰 — 무엇을 보고 어떻게 승인·변경요청하나 — 실습 Lab

INCIDENT RESPONSE

0 / 5 단계 완료

📚 PREREQUISITES

Labsweng-branch-strategy

Theorysw-engineering/code-review-pr

TRACK

SW-ENGINEERING

SLA

50분

LEVEL

중급

PHASES

3단계

ENV

local

INCOMING TICKET

“팀 시니어: "PR 리뷰가 형식만 남았어요. 어떤 사람은 500줄짜리 PR에 LGTM만 달고 승인하고, 어떤 사람은 변수명·세미콜론만 잔뜩 지적하면서 정작 결제 금액 계산이 틀린 건 그냥 통과시켜요. 무엇을 어떤 순서로 보고, 어떻게 코멘트를 달고, 언제 승인하고 언제 변경을 요청하는지 — 우리 팀 리뷰 기준을 잡아 주세요."”

YOUR ROLE

코드리뷰 문화를 세우는 개발 리드/리뷰어

안 하면 나중에

리뷰가 형식만 남으면 로직 결함이 리뷰를 통과해 프로덕션으로 새고, 스타일 트집으로 감정이 상하며, 리뷰가 지식 공유의 기회가 되지 못한다.

📋상황 브리핑

팀 시니어가 말합니다.

"우리 PR 리뷰가 형식만 남았어요. 어떤 사람은 500줄짜리 PR에 'LGTM' 한 줄 달고 승인하고, 어떤 사람은 세미콜론이랑 변수명만 잔뜩 지적하면서 정작 결제 금액 계산이 반올림에서 틀린 건 그냥 통과시켜요. 리뷰가 버그를 못 잡고, 서로 감정만 상해요. 무엇을 어떤 순서로 보고, 어떻게 코멘트를 달고, 언제 승인하고 언제 변경을 요청하는지 — 우리 팀 리뷰 기준을 잡아 주세요."

코드리뷰는 '트집 잡기'가 아니라 '변경이 안전하게 합쳐지도록 최소 한 쌍의 눈을 보태는 일'이다. 이 Lab은 PR을 리뷰 가능한 크기로 만드는 판단부터, diff를 보는 우선순위, 존중하는 코멘트와 승인/변경요청 결정까지 리뷰의 실무 기준을 세운다.

⏱ 50분📊 중급🔧 3단계#code-review#pull-request#pr#quality

MISSION

PR 크기·범위 판단 — 이 diff는 리뷰 가능한가

변경 규모와 성격을 진단해 이 PR이 한 번에 제대로 리뷰될 수 있는지 판정하고, 너무 크면 어떤 축으로 쪼갤지 제안한다.

리뷰의 렌즈 — 무엇을 어떤 우선순위로 보나

diff를 열었을 때 정확성·경계조건부터 스타일까지 무엇을 어떤 순서로 볼지 정하고, 보안·시크릿을 상시 렌즈로 고정해 "스타일만 보고 로직을 놓치는" 실패를 막는다.

리뷰 코멘트와 결정 — 차단·제안 구분과 approve/request-changes

코멘트에 차단(blocking)과 제안(nit)을 구분해 표기하고, 코드를 주어로 존중하는 어조로 쓰며, blocking 이슈 유무로 approve와 request-changes를 규칙으로 결정한다.

📌 선수 지식

• [실습] sweng-branch-strategy

• [이론] sw-engineering/code-review-pr