Addressed in a1b2c3d: 리뷰 지적을 어느 커밋에서 고쳤는지 알리기

오늘의 영어 표현 17/30 · 단계: 리뷰 받기·답하기. 예문은 모두 이 글을 위해 지어낸 것입니다. 실제 저장소의 댓글을 옮긴 것이 아니며, 커밋 해시는 예시입니다.

한눈에 보기
1. 뜻과 뉘앙스
2. 예문
3. 비슷한 표현과 차이
4. 한국인이 자주 하는 실수
5. 한 줄 연습
모범 답안
참고

한눈에 보기


표현	Addressed in a1b2c3d. / I've addressed all the comments.
뜻	a1b2c3d 커밋에서 반영했습니다 / 모든 지적을 반영했습니다.
쓰는 자리	리뷰 줄 댓글의 답, PR 댓글
공손함	중립. 보고하는 말입니다

1. 뜻과 뉘앙스

address 는 「주소」라는 명사 말고도 「(문제나 질문을) 다루다, 처리하다」라는 동사로 자주 씁니다. address a comment 는 「리뷰 지적을 반영하다」, address a concern 은 「우려에 대응하다」입니다. fix 는 「고치다」라서 버그에만 어울리지만, address 는 질문에 답하거나, 문서를 보충하거나, 설계를 바꾸는 일까지 넓게 덮습니다.

Addressed in (커밋) 은 「어디서 반영했는지」를 알려 줍니다. GitHub 는 댓글 속 커밋 해시를 자동으로 링크로 바꿔 주므로, 리뷰어는 그 링크를 눌러 해당 변경만 볼 수 있습니다. 여러 지적을 반영한 경우 줄 댓글마다 이렇게 답해 두면 리뷰어가 하나씩 확인하고 대화를 Resolve 할 수 있습니다.

커밋을 따로 남기는 것과도 관련이 있습니다. Kubernetes 기여 가이드는 리뷰 중에는 수정 사항을 새 커밋으로 올리고 아직 스쿼시하지 말라고 안내합니다. 리뷰어가 수정분만 따로 볼 수 있게 하기 위해서입니다. 스쿼시는 리뷰가 끝날 무렵 요청을 받았을 때 합니다(21편). 스쿼시나 rebase 를 하면 해시가 바뀌므로, 그 뒤에는 Addressed in 링크가 옛 커밋을 가리킬 수 있다는 점도 알아 둡니다.

대화를 누가 Resolve 할지는 저장소와 리뷰어마다 습관이 다릅니다. 리뷰어가 직접 확인하고 닫기를 원하는 경우가 많으므로, 확신이 없으면 작성자는 답만 달고 닫는 일은 리뷰어에게 맡깁니다.

2. 예문

상황 1: 줄 댓글에 답할 때

Addressed in a1b2c3d. I switched to errors.Is() as you suggested.

상황 2: 여러 지적을 한 번에 반영했을 때

Thanks for the thorough review! I've addressed all comments in the last two commits:

renamed cfg to config

added a nil check before dereferencing spec.template

documented the new flag in the README

thorough 는 「꼼꼼한」입니다. 리뷰어의 수고를 인정하는 말로 자주 씁니다.

상황 3: 일부만 반영했을 때

Addressed most of the comments in d4e5f6a. For the caching suggestion, I left a reply below; I'm not sure it's worth the added complexity here.

모두 반영하지 않았다면 어떤 것이 남았는지 분명히 밝힙니다.

3. 비슷한 표현과 차이

Done. / Fixed. 가장 짧은 답입니다. 사소한 지적(nit)에는 이것으로 충분합니다.
Updated. 「수정했다」는 넓은 뜻입니다. 문서나 설명을 바꿨을 때 자연스럽습니다.
Resolved. 대화 자체가 해결됐다는 뜻으로, GitHub 의 Resolve conversation 버튼과 같은 말입니다.

4. 한국인이 자주 하는 실수

I reflected your comment. 「반영하다」를 reflect 로 직역하면 「거울에 비추다」나 「곰곰이 생각하다」로 읽혀 어색합니다.
- 고친 문장: I've addressed your comment. 또는 I've applied your suggestion.
I modified as you said. modify 의 목적어가 빠졌고 as you said 는 다소 딱딱합니다.
- 고친 문장: Updated as suggested.
All fixed. Please merge. 리뷰어가 확인하기 전에 병합을 요청하는 것은 이릅니다.
- 고친 문장: I've addressed all the comments. PTAL.

5. 한 줄 연습

다음 한국어 문장을 줄 댓글의 답으로 바꿔 쓰십시오.

d4e5f6a 커밋에서 반영했습니다. 제안해 주신 대로 타임아웃을 설정값으로 뺐습니다.

모범 답안

Addressed in d4e5f6a. I made the timeout configurable as you suggested.

「설정값으로 빼다」는 make ... configurable 이 가장 자연스럽습니다. extract to config 라고 써도 뜻은 통합니다.

티티 한마디. 「반영했습니다」 뒤에 커밋 해시 하나만 붙여도 리뷰어의 클릭 수가 줄어듭니다.

참고

Merriam-Webster, 「address」(동사): https://www.merriam-webster.com/dictionary/address
Kubernetes 기여 가이드, Pull Request Process(리뷰 중 스쿼시하지 않기): https://github.com/kubernetes/community/blob/master/contributors/guide/pull-requests.md
GitHub Docs, Autolinked references and URLs(커밋 해시 링크): https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/autolinked-references-and-urls