[코드리뷰]

"코드 리뷰는 지적이 아닌 커뮤니케이션이다."

코드리뷰를 통한 프로젝트는 코드를 견고하고 안전하게 작성할 수 있고,
팀 지식 공유를 통해 성장할 수 있고, 팀 개발 표준을 통한 확장성을 커지는 장점을 얻을 수 있다.

또한 코드 작성자는 자신의 코드를 다시 확인하고 새로운 접근 방법을 발견할 기획을 얻을 수 있으며, 리뷰어는 코드를 통해 질문을 할 수 있다.

코드리뷰를 하게되면 작업과 배포가 느려지는 경우가 있을 수 있어 꺼려하는 경우가 있으나 코드리뷰를 하지 않아 발생하는 문제로 인해 개발이 더 늦어질 수 있다.

정성을 다하는 커뮤니케이션 태도

코드 리뷰가 공격적으로 느껴지지 않게 하는 방법은 정성을하다는 커뮤니케이션이라고 한다.

1. "You" 대신 "We"를 사용하기 : 상대를 리뷰하는게 아닌 함께 코드를 바꿔보자라는 태도로 "We"를 사용하자.
2. 형용사/부사를 사용하지 않기 : 큰 실수를 한 것처럼 느낄 수 있기 때문에 조심해야한다.
3. 대문자로만 이야기하기 않기 : 너무 강하게 들릴 수 있다.

코드 작성자도 코드 리뷰에 대한 감사함과 담담하게 대하는 태도를 가져야 한다.

코드리뷰 잘하기

1. 빠르게 리뷰하기

   • 리뷰 시간을 체크하고 코드리뷰 알림을 보내는 workerb for pull request 사용하기
   • 커밋 시 우선순위 정하기

2. nit 줄이기

   • 사소한 nit은 코드리뷰에 포함시키지 않는 것이 좋다.
   • Jenkins나 github action tool 사용

   nit : 큰 맥락에서는 중요하지 않지만 품질 향상을 위해 사용하는 것

3. 변경 사이즈 줄이기
   너무 많은 변경을 한번에 커밋하게 되면 리뷰어 입장에서 시간이 오래걸리는 악순환이 생길 수 있다.
   변경 사항을 줄려 리뷰어를 배려하자.

---

참조 - 코드리뷰 전문가 서지연의 강연