반대 의견에 대처하는 방법

작성자가 당신의 의견에 반대할 수도 있고, 또는 너무 엄격한 잣대를 적용한다며 불평할 수도 있다.

누구 말이 맞는가?

작성자가 당신의 의견에 반대한다면 잠시 시간을 갖고 그의 말이 맞는지 고민해본다. 그럼에도 당신의 말이 맞다는 생각이 들면 추가적인 설명을 하되 작성자의 의견도 충분히 이해했다는 점을 밝힌다. 코드 품질은 매우 천천히 조금씩 증가한다. 당신의 제안이 코드 품질을 증가시킨다면 더욱 적극적으로 의견을 피력해야 한다. 그리고 언제나 친절하게 상대를 대한다.

작성자의 기분이 상하지 않을까

여러 리뷰어들은 자신이 코드 수정을 요구하면 작성자의 기분이 상하지 않을까 걱정한다. 실제로 기분이 상하는 사람도 있긴 하지만 일시적일 뿐이고 금세 고마워한다. 그리고 대체로 개발자가 기분이 상하는 이유는 예의없는 말투 때문이다. 항상 친절하게 의견을 나눈다.

“나중에 다듬을게요”

흔히들 일을 빨리 마무리 짓고 싶어하기 때문에 사소한 코드 정리를 미루는 경우가 있다. 모두가 이해하는 바이다. 그래서 이번 CL을 통과시켜주면 다음 CL에 꼭 반영하겠다고 약속을 한다. 대부분의 개발자들은 금세 새로운 CL을 만들어서 올리기도 하지만 우리의 경험으로 보건대 곧바로 후속 CL을 작업하지 않으면 사실상 영영 잊혀진다. 사람들이 무책임해서가 아니라 워낙 일이 많기 때문이다. 이런 작업을 미루다보면 코드 품질이 퇴보한다.

기준이 너무 까다롭다는 불평

느슨하게 코드 리뷰를 하던 팀에서 엄격한 리뷰를 도입하면 이곳 저곳에서 불평의 목소리가 커진다. 몇 달이 걸리기도 하지만 코드 리뷰의 속도가 점차 빨라지면 불평이 사그라들기도 한다. 그러다가 자신들이 생산하고 유지 보수하는 코드의 품질이 나아지는걸 보면서 처음에 불평했던 사람들이 더 열성적인 지지자가 되곤 한다.

의견 충돌 해결하기

위의 가이드를 다 지켰는데도 작성자와 합의를 도출할 수 없다면 코드 리뷰의 대원칙 부분을 참고하여 갈등을 해소한다.