소개

코드 리뷰를 받고 있을 때, 이것은 수정이 필수적인 리뷰 코멘트인가, 가능하면 수정하는 것이 좋은 것인가, 판단에 헤매는 때가 없습니까?

사귀는 긴 팀이나, 멤버끼리 리뷰를 하고 있다고 감각적으로 알 수도 있지만, 새로운 팀, 새로운 멤버 등 관계성이 완성되기 전이라면, 온도감을 모르고 판단에 헤매는 일이 있을 것 같다 입니다.

결과, 모든 것을 대응하려고 너무 노력해 버려, 수정에 무섭게 시간이 걸려 릴리스가 지연되는 등 본말 전도한 불행한 케이스도 일어날 수 있을 것 같습니다.



예) 선배가 말하고 있고 모두 고쳐야 한다...!

리뷰 댓글에 라벨 추가

[must] 반드시 대응해 주었으면 한다!

[imo] 자신의 의견이나 제안· 기호. 자신이라면 이렇게 쓰지만 어떨까요?

[nits] 사소한 지적. 단지 작은 지적이지만 가능하면 고쳐주세요. 들여 쓰기 및 타이포

[ask] 질문, 확인. 코드의 의미와 배경을 모르기 때문에 알려주세요! 이런 이해로 OK?

가능하면 고치고 싶지만, 이번은 릴리스 우선해 다른 PR로 대응하게 해 주자!

이런 느낌으로 리뷰 코멘트에 라벨이 붙어 있는 것만으로, 온도감이나 정보의 전달성을 알기 쉬워집니다.
온도감을 알면 판단이나 상담도 쉬워지고, 단지 작은 궁리만으로 개발 속도가 오를 것 같습니다.

요약

조사해 보면 일반적입니다.
리뷰 댓글에 라벨만 있으면 검토 비용이 낮아집니다. 지금 실천할 수 있는 내용이므로 팀에서 실천해 보세요.

신인 프로그래머, 선배 프로그래머 여러분이 행복한 시작을 할 수 있도록.

참고 링크

나쁜 코드를 미워하고 사람을 미워하지! Pulllike를 보내기 전에 유념하고 싶은 코드 리뷰 커뮤니케이션 기술

pull request를 이용한 개발 워크 플로

【오늘부터 가능하다】 커밋 메시지에 「접두사」를 붙이는 것만으로 개발 효율이 높아진 이야기