쓰기 · 무료 레슨
코드 리뷰 영어: 필수 수정인가요, 제안인가요?
동료가 리뷰 댓글 다섯 개를 보고 모두 해결해야 병합할 수 있다고 생각합니다. 필수 수정과 나중에 해도 되는 개선을 구별해 주세요.
Scott Mendoza 교수핵심 정리
상태를 직접 쓰세요. Required before merge: handle the empty input. Optional: consider a clearer helper name. 정중한 말투만으로는 필수 수정인지 알 수 없습니다.
필수 여부와 정중함을 구별하세요.
Could you handle the empty input?는 필수 수정을 정중하게 요청하는 말일 수 있습니다. Consider renaming this helper는 개선 제안이지만 팀의 규칙도 중요합니다. 의도가 불분명할 수 있다면 required before merge나 optional을 붙이고 구체적인 이유를 짧게 설명하세요.
예문 비교
Required before merge: please handle the empty input.
Optional: consider a clearer helper name.
Is this required before merge, or is it an optional suggestion?
non-blocking은 보통 팀 절차상 이 댓글 때문에 진행을 막으려는 것이 아니라는 뜻입니다. 쓸모없는 의견이라는 뜻도 아니고 다른 승인이나 실패한 검사를 무시해도 된다는 뜻도 아닙니다.
뜻을 단계별로 구성하기
리뷰 상태 명시
Required before merge:
요청 행동 제시
Please handle the empty input.
별도의 선택 제안 표시
Optional: consider a clearer helper name.
지금 연습해 보세요
리뷰어의 의도 읽기
전체 정답과 해설 보기
댓글: Optional: consider renaming this helper. 표시에 맞는 해석은 무엇인가요?
This comment proposes an optional improvement.optional은 이 제안의 성격입니다. 이름 변경을 필수로 만들거나 다른 검사를 없애지 않습니다.
댓글: Required before merge: could you handle the empty input? could you 때문에 선택 사항인가요?
No, the explicit required label controls the intent.정중함과 필수 여부는 다릅니다. 표시에 요구 사항이 명시되어 있습니다.
Could we rename this?라는 댓글에 표시나 공유된 규칙이 없습니다. 상태를 확인하는 답은 무엇인가요?
Is this required before merge, or an optional suggestion?의문문 형태만 보고 추측하지 말고 필수 여부를 물으세요.
직접 연습하기
두 리뷰 의견 구별하기
이번 리뷰에서는 빈 입력 처리가 병합 전 필수이고 보조 함수 이름 변경은 선택입니다. 작성자가 구별하도록 댓글을 고치세요.
Please handle empty input and rename the helper.
예시 답변 보기
Required before merge: handle empty input. Optional: rename the helper for clarity.
실제 팀 리뷰 절차를 따르세요. 이 표시는 댓글을 설명하며 병합 권한을 주는 것이 아닙니다.
조금 더 알아보기
자주 묻는 질문
다음으로 넘어가기 전에 알아두면 좋은 내용입니다.
nit는 항상 선택 사항인가요?
팀 전체에 통하는 규칙은 아닙니다. 작은 지적을 뜻하는 경우가 많지만 작다는 것과 선택이라는 것은 다릅니다. 팀에서의 의미가 불분명하면 물으세요.
Is this small cleanup required in this change?
표시만 하고 설명은 없어도 되나요?
표시는 상태를 알려 주지만 짧은 이유도 변경을 이해하는 데 도움이 됩니다. 사람이 아니라 작업 내용을 대상으로 말하세요.
Optional: a clearer name would make the helper’s purpose easier to recognize.
일상에서 사용해 보세요
필수와 선택을 밝히고 댓글이 허용하지 않는 범위도 지키세요.
직무별 영어 · 출시 예정
소프트웨어 리뷰 의사소통을 더 분명하게 해 보세요.
English for Software Engineers: Career Simulation 과정은 출시 예정입니다. 이 과정은 요구 사항, 리뷰, 테스트, 출시 논의에 필요한 영어를 다룰 예정입니다. 과정 페이지에서 출시 소식을 신청하세요.
출시 소식 받기소프트웨어 엔지니어 · 직무 시뮬레이션
다음 학습 · Udemy에서 수강 가능
영어 기초도 함께 다져 보세요.
현재 수강 가능한 English Grammar and English Speaking Master Course는 문법, 문장 만들기, 의사소통의 기초를 다룹니다. 분명한 요청과 설명은 리뷰 의도를 전달하는 데 도움이 됩니다.
강좌 살펴보기 English Master Course: 영문법과 영어 말하기