코드 리뷰 가이드
PR에 있는 답안 코드가 저장소의 기본 브랜치로 병합이 되려면 상호 코드 리뷰를 하고 PR을 승인하는 과정이 필요합니다. 물론 코치 분들이 코드 리뷰를 많이 해주시겠지만 다른 멤버들이 제출한 다양한 답안 코드를 보면서 문제에 접근하는 새로운 관점과 통찰을 얻을 수 있기 때문에 맴버 분들에게도 코드 리뷰를 적극 권장합니다.
답안 코드가 너무 오랫동안 PR 상태로 방치되는 것을 방지하기 위해서 코드 리뷰 책임제를 도입하였습니다. 본인 코가 아무리 석자시더라도 자기 PR 바로 다음에 올라온 PR은 반드시 책임감을 갖고 리뷰를 해주셔야 합니다! 자신에게 할당된 PR 뿐만 아니라 다른 PR에 대한 코드 리뷰도 당연히 언제나 환영합니다.
코드 리뷰 관련해서 많은 분들이 오해하시는 부분이 있는데, 코드 리뷰를 통해 반드시 더 나은 해결책을 제시하거나 버그나 실수를 찾아주셔야 하는 것은 아닙니다. 코드를 보고 이해가 잘 안 되는 부분이 있으면 아래 캡쳐처럼 질문을 남기는 것도 매우 좋은 코드 리뷰 방법입니다. 그러면 코드 작성자는 본인의 코드가 얼마나 읽기 쉬운지에 대해서 다시 한 번 고민해볼 수 있습니다. 코딩 면접에서 가독성이 좋은 코드를 짜는 것도 매우 중요한 부분입니다.
