안녕하세요! 오랜만에 인사드립니다. 2월엔 이런저런 신경쓸게 너무 많아서 글 업로드를 못했는데요.. (뭔 얼마나 바빴길래 글도 못썼냐라고 생각하시면 맞습니다.. 핑계 같네요..ㅎㅎ) 오랜만에 쓰는 글은 코드리뷰에 관한 글을 가져왔습니다. 아래는 카카오 테크에서 작년 이맘때쯤 기고된 코드리뷰에 관한 글인데요. 글 자체는 아래를 읽어보시고 아래 글은 좀 이제 시니어가 교육생한테 어떻게 리뷰하면 좋은지에 대해 초점이 맞춰져 있어 저는 이 글을 같이 일하는 동료, 팀원분들과 하는 코드리뷰에 관한 관점에서 글을 다시 바라보려고 합니다. 글에서는 5가지 규칙에 대해 이야기하고 있는데요! 이에 대해 제 해석을 다시 살펴보시죠. 1. 왜 개선이 필요한지 이유를 충분한 설명해 주세요. 이 규칙에 대해선 다른 관점이 필요도 없이 온전히 공감합니다. 섹션에 마지막에 있는 개선의 필요성에 대한 설명했다기 보단 팀원들 간의 코드리뷰에선 본인이 코드리뷰를 달면서 어떤 생각으로 이런 리뷰를 달았는지 의도를 명확히 전달하는게 좋다고 생각합니다. 2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요. 이 규칙은 말을 '코드에 정답은 없으므로 다양한 관점을 가지고 본인의 생각에 대해 이야기하고 제안하자.' 정도로 바꾸고 싶네요! 코드에 정답은 없지만 여러 모범 답안이 있을 수 있습니다. 언제나 더 나은 방법이 있을 수 있으니 작업자와 이런 저런 의견을 주고 받는 것은 아주 건설적인 논의라고 생각하고 그를 통해 서로 발전도 할 수 있다고 생각합니다. 해당 의견을 채택할지 말지는 작업자의 몫이고 채택되지 않더라도 좋은 논의가 있었다면 그 자체로도 만족스럽습니다. 3. 코드를 클린 하게 유지하고, 일관되게 구현하도록 안내해 주세요. 이 규칙도 안내라는 말만 좀 바꾼다면 그대로 가져가고 싶네요. 코드리뷰에 여러 관점이 있지만 팀의 컨벤션과 코드 내의 일관성은 중요합니다. 혹시나 클린하게 유지라는 말이 무슨 말일까 고민된다면 클린 코드라는 책을 읽어보신다면 클린하게 라는 의미에 대해 조금 더 이해하실 수 있지 않을까 싶네요. 4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요. 이 말은 해당 섹션은 첫번째 문장인 '코드 리뷰를 하는 목적은 서로 생각을 공유하고 좀 더 깔끔한 코드를 작성하기 위함입니다.' 로 대체하고 싶습니다. 글에서는 리뷰를 받는 사람이 숙제 같이 느껴지지 않게 해라고 말했지만 필드에서 저는 서로 숙제같이 느껴지지 않는 것이 중요하다고 생각합니다. 무슨 말이냐하면 현업에 종사하시는 여러분! 코드 리뷰를 받는거 보단 하는 것이 숙제처럼 느껴지고 리뷰해야하는데.. 하면서 좀 미루신 경험 없으신가요!? (저만 그런가요 크흠흠.. 그럼 반성을..) 리뷰어든 리뷰이든 더 나은 프로덕트를 위해 코드리뷰는 반드시 필요합니다. 출근하면 앞에 30분은 코드 리뷰 같은 룰이라던지 데브옵스의 도움을 받는다던지 등 그라운드 룰이 있어도 좋을거 같다는 생각이 드네요! 5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요. 이 규칙도 아주 공감합니다. 코드에 고칠거나 제안할 내용이 없더라도 고생한 동료분들을 위해 수고하셨다라는 내용이라던지 최소한 LGTM이라도 남겨주시면 어떨까요?