오늘의 영어 표현 19/30 · 단계: 리뷰 받기·답하기. 예문은 모두 이 글을 위해 지어낸 것입니다. 실제 저장소의 댓글을 옮긴 것이 아니며, 함수 이름은 예시입니다.
목차
한눈에 보기
| 표현 | Not a blocker, but ... / (non-blocking) |
| 뜻 | 병합을 막는 지적은 아니지만 ... |
| 쓰는 자리 | 리뷰 댓글의 머리말이나 꼬리말, 회의 |
| 공손함 | 부드러움. 상대의 부담을 덜어 줍니다 |
1. 뜻과 뉘앙스
block 은 「막다」, blocker 는 「(일의 진행을) 막는 것」입니다. 개발 현장에서 blocker 는 「이것이 해결되지 않으면 다음 단계로 갈 수 없는 문제」를 뜻합니다. 릴리스 blocker, 병합 blocker 처럼 씁니다. 그래서 Not a blocker 는 「이 지적은 고치지 않아도 병합할 수 있다」는 뜻입니다.
리뷰 댓글에는 무게가 서로 다른 것들이 섞입니다. 반드시 고쳐야 할 버그, 고치면 좋은 개선, 단순한 참고 정보입니다. 이것이 구분되지 않으면 작성자는 모든 댓글을 필수로 받아들여 PR 이 끝없이 길어집니다. Google 의 코드 리뷰 지침은 그래서 댓글에 Nit, Optional(또는 Consider), FYI 같은 머리말을 달아 무게를 밝히라고 권합니다. Not a blocker 는 이 머리말들과 같은 역할을 하는 일상 표현입니다.
리뷰어가 쓸 때는 대개 but 으로 제안을 잇습니다. Not a blocker, but it might be worth ... 처럼 「막지는 않지만 고려해 볼 만하다」는 구조입니다. 작성자가 쓸 때는 「이것이 막는 문제인가요?」라고 물어 우선순위를 확인합니다. 이 질문은 무례하지 않습니다. 오히려 리뷰어가 의도를 분명히 할 기회를 줍니다.
2. 예문
상황 1: 리뷰어의 제안
Not a blocker, but it might be worth adding a short comment explaining why we retry three times.
it might be worth -ing 는 「~할 만한 가치가 있을지도 모른다」로, 제안을 부드럽게 만드는 단골 표현입니다.
상황 2: 머리말로 달 때
(non-blocking) This could be simplified with
slices.Contains(), but the current version is fine too.
괄호 안의 non-blocking 은 리뷰 댓글 첫머리에 붙여 쓰는 경우가 많습니다.
상황 3: 작성자가 우선순위를 물을 때
Thanks for the suggestion! Is this a blocker for this PR, or would you be okay with handling it in a follow-up?
blocker 인지 묻고, 아니라면 후속 PR 로 넘기자는 대안을 함께 제시합니다(10편).
3. 비슷한 표현과 차이
- Optional: ... Google 지침의 머리말입니다. 「좋은 생각이지만 꼭 필요하지는 않다」는 뜻입니다.
- Nit: ... 사소한 지적입니다(14편). 고치는 편이 맞지만 영향이 작습니다. Not a blocker 보다 범위가 좁습니다.
- This is a must-fix. / Blocking. 반대로 반드시 고쳐야 한다는 표시입니다. 보안 문제나 데이터 손실 위험처럼 분명한 이유가 있을 때 씁니다.
4. 한국인이 자주 하는 실수
- 리뷰어가 Not a blocker 라고 했는데 반드시 고쳐야 하는 것으로 받아들여 PR 을 크게 늘리기. 고칠지 말지는 작성자가 정해도 됩니다. 고치지 않는다면 짧게 답합니다.
- 고친 문장: Good idea. I'll leave it for now to keep this PR small.
- Is it blocking? 짧아서 다소 퉁명스럽게 들릴 수 있습니다.
- 고친 문장: Just to clarify, is this a blocker for merging?
- This is not block. block 은 동사이자 명사지만 이 자리에는 blocker 나 blocking 이 맞습니다.
- 고친 문장: This is not blocking.
5. 한 줄 연습
리뷰어 입장에서 다음 내용을 댓글로 쓰십시오.
병합을 막는 건 아닌데요, 이 로그 수준을 debug 로 낮추면 어떨까요?
모범 답안
Not a blocker, but what do you think about lowering this log level to debug?
what do you think about -ing? 는 제안을 질문으로 바꿔 작성자에게 결정을 맡기는 표현입니다. Could we lower ...? 라고 써도 자연스럽습니다.
티티 한마디. 모든 지적이 같은 무게는 아닙니다. 무게를 한 단어로 붙여 주면 작성자는 어디에 힘을 쓸지 압니다.
참고
- Google Engineering Practices, How to write code review comments(Label comment severity): https://google.github.io/eng-practices/review/reviewer/comments.html
- Merriam-Webster, 「blocker」: https://www.merriam-webster.com/dictionary/blocker