오늘의 영어 표현 15/30 · 단계: 리뷰 받기·답하기. 예문은 모두 이 글을 위해 지어낸 것입니다. 실제 저장소의 댓글을 옮긴 것이 아니며, 사람과 번호는 예시입니다.
목차
한눈에 보기
| 표현 | LGTM / LGTM with one nit / /lgtm |
| 뜻 | 좋아 보입니다(승인합니다) / 사소한 것 하나만 빼면 좋습니다 |
| 쓰는 자리 | 리뷰 결론, Prow 명령 |
| 공손함 | 중립. 리뷰의 결론을 알리는 말입니다 |
1. 뜻과 뉘앙스
LGTM 은 Looks Good To Me 의 줄임말입니다. LLVM 의 용어집은 LGTM 을 「리뷰어가 그 패치를 커밋해도 괜찮다고 본다」는 뜻으로 설명하고, LLVM 코드 리뷰 문서는 커밋 승인이 보통 LGTM 이 담긴 메시지로 표시된다고 적습니다. 같은 문서는 확신이 없는 리뷰어에게 「LGTM, but please wait for (다른 리뷰어)」처럼 조건을 붙인 승인을 하라고 권합니다.
to me 라는 말에 주목합니다. 「나에게는 좋아 보인다」이므로, 다른 리뷰어나 메인테이너의 판단은 따로 남아 있을 수 있습니다. 그래서 LGTM 한 번이 곧 병합을 뜻하지는 않습니다.
Kubernetes 와 Prow 의 /lgtm, /approve. Kubernetes 계열 저장소에서는 댓글에 명령을 적으면 Prow 봇이 라벨을 붙입니다. Kubernetes 의 OWNERS 문서에 따르면 다음과 같습니다.
/lgtm: 리뷰어가 코드의 품질, 정확성, 스타일을 보고 좋다고 판단했다는 표시입니다. 봇이lgtm라벨을 붙입니다. GitHub 조직의 구성원이면 리뷰어가 될 수 있지만, PR 작성자는 자기 PR 에 /lgtm 을 달 수 없습니다. 취소는/lgtm cancel입니다./approve: 의존성, 호환성, API 와 플래그 정의 같은 더 넓은 승인 기준을 승인자가 확인했다는 표시입니다. 해당 디렉터리의 OWNERS 파일에 이름이 있는 사람만 쓸 수 있고, 필요한 OWNERS 마다 승인이 모이면approved라벨이 붙습니다.- 병합 자동화는 대개
lgtm과approved두 라벨이 모두 있고,do-not-merge/hold같은 막는 라벨이 없을 때 병합합니다.
그러니 Kubernetes 계열 저장소에서 「LGTM 을 받았다」는 말과 「병합 가능하다」는 말은 다릅니다. 승인자의 /approve 가 따로 필요할 수 있습니다.
2. 예문
상황 1: 리뷰어의 조건부 승인
LGTM with one nit. Feel free to merge after fixing the typo in the comment.
상황 2: Prow 를 쓰는 저장소에서
Thanks for the quick turnaround. The changes look good to me.
/lgtm
quick turnaround 는 「빠른 반영」이라는 뜻으로, 리뷰를 빨리 반영한 작성자를 칭찬하는 말입니다.
상황 3: 작성자가 LGTM 을 받은 뒤
Thanks for the review! I've fixed the nit. Do I need an approval from someone in OWNERS, or is this good to go?
good to go 는 「진행해도 된다」는 뜻의 구어입니다.
3. 비슷한 표현과 차이
- Approve(GitHub 리뷰 기능). GitHub 의 Approve 버튼은 저장소 설정에 따라 병합 조건이 되기도 합니다. Prow 의 /approve 와는 다른 장치입니다.
- Looks good overall, but ... 「전체로는 좋지만」으로, 아직 승인이 아니라 고칠 것이 남았다는 뜻입니다.
- SGTM(Sounds Good To Me). 코드가 아니라 제안이나 계획에 동의할 때 씁니다.
4. 한국인이 자주 하는 실수
- 작성자가 자기 PR 에 /lgtm 을 다는 것. 규칙상 허용되지 않고 봇이 거절합니다.
- 고친 문장: Thanks! Could someone from OWNERS take a look for approval?
- Thank you very very much for your LGTM!! 감사는 좋지만 지나친 강조와 느낌표는 업무 대화에서 어색합니다.
- 고친 문장: Thanks for the review!
- LGTM 을 받자마자 「Please merge」라고 재촉하기. 다른 리뷰어나 승인자의 판단이 남아 있을 수 있습니다.
- 고친 문장: Thanks! Is there anything else needed before this can be merged?
5. 한 줄 연습
리뷰어 입장에서 다음 내용을 댓글로 쓰십시오.
전체적으로 좋습니다. 테스트 이름 오타 하나만 고쳐 주시면 됩니다.
모범 답안
LGTM with one nit: there's a typo in the test name.
LGTM with one nit 이라는 말 안에 「승인」과 「사소한 수정 하나」가 함께 들어 있습니다. 무엇이 그 하나인지는 콜론 뒤에 짧게 적습니다.
티티 한마디. LGTM 은 「나에게는」 좋다는 뜻입니다. 병합까지 무엇이 남았는지는 저장소의 규칙을 봅니다.
참고
- Kubernetes 기여 가이드, OWNERS files(/lgtm, /approve): https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md
- LLVM Lexicon, 「LGTM」: https://llvm.org/docs/Lexicon.html
- LLVM Code-Review Policy and Practices: https://llvm.org/docs/CodeReview.html