Re- 4Alien 팀은 코드 리뷰를 중요하게 생각합니다.

매주 회고에서 한 주의 코드 리뷰에 대해서 회고합니다.

리뷰가 공격적이진 않았는 지 리뷰할 코드를 꼼꼼히 읽어보며 작성 했는 지 회고합니다.

코드 리뷰의 의미는 팀원이 코드를 이해하고 미처 생각하지 못한 부분이 있다면 리뷰를 주고 받으며 서로의 코드 스타일을 이해해가는 데에 있다고 생각합니다.

현재는 request한 사람들 중 3 approve 이상 있을 때에 dev나 main 브랜치로 merge할 수 있습니다.

리뷰가 빠르게 필요하다면 Slack DM으로 리뷰를 요청하면 팀원이 최대한 빠르게 코드 리뷰를 남깁니다.

1. 왜 개선이 필요한지 이유를 충분한 설명해 주세요.

리뷰어가 코드 개선의 필요성을 느끼고 리뷰를 남긴다면 충분한 이유가 뒷받침되어야 합니다. 만약 코드 리뷰가 주관적이거나 추상적이라면 리뷰이는 혼란을 느낄 수 있습니다. 그래서 피드백을 작성할 때 리뷰이가 개선의 필요성을 느낄 수 있도록 구체적인 이유를 작성하면 좋습니다.

2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.

업무와 코드 리뷰를 병행하는 건 쉽지 않은 일입니다. 업무량은 많은데 코드 리뷰까지 신경 쓰려고 하다 보니 상세히 코드를 살펴보지 못하는 경우도 생기는데요, 그럴 때 리뷰어는 “000로 변경하세요”, “000을 쓰세요”와 같이 현재 코드의 개선점을 찾아 답을 알려주는 리뷰가 늘어나기도 합니다. 이러한 리뷰가 늘어날수록 리뷰이는 주어진 리뷰의 코드만 변경할 뿐 스스로 고민하는 시간은 줄어듭니다. 답을 알려주는 리뷰가 많아질수록 코드에 대한 애정도 사라지고 수동적인 개발자가 될 수 있으므로 스스로 고민하고 개선 방법을 찾는 것이 중요합니다.

3. 코드를 클린 하게 유지하고, 일관되게 구현하도록 안내해 주세요.

‘클린 코드’는 개발자라면 한 번쯤 고민해 봤을 만한 주제입니다. 코드의 품질을 높이기 위해 개발자들은 항상 깨끗하게 코드를 유지하려고 노력을 많이 하는데요. 코드 가독성이 높아지고 일의 생산성도 향상되기 때문에 코드 리뷰를 할 때도 코드를 깨끗하게 유지하는 방법을 고민하는 것은 중요합니다.

4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.

코드 리뷰를 하는 목적은 서로 생각을 공유하고 좀 더 깔끔한 코드를 작성하기 위함입니다. 그러나 코드 리뷰를 받아본 경험이 적은 개발자에게는 리뷰가 숙제로 받아들일 수도 있습니다. 이러한 상황을 예방하기 위해 리뷰이가 코드리뷰를 학습 일부로 받아들이기 위해 피드백을 자주 주고받고, 성장하고 있다는 생각을 심어주는 것이 좋습니다.