Git (Bitbucket) 코드 리뷰 및 PR 설정

저번 블로그에서 저희 회사 빡센 코드 리뷰에 대해 이야기를 했으니, 이번 블로그에서는 어떻게 Bitbucket를 이용하여 코드 리뷰를 할 수 있는 개발 환경을 만들 수 있는지 알아보도록 하겠습니다.

우선 Bitbucket.org에서 레파지토리를 하나 만든 후 Settings로 이동하여 User and group access를 클릭합니다.

기본적으로 Developers라는 그룹을 제공하고 있습니다. 이 그룹은 반드시 WRITE 권한을 가지고 있어야 합니다. 이제 이 그룹을 클릭하여 사용자를 추가합니다. 제가 개인적으로 진행하는 프로젝트에는 4명의 개발자가 할당되어 있습니다.

저장이 완료되면 이 개발자들은 현재 메인 레파지토리에 바로 쓰기 권한을 가지고 있습니다. 이 레파지토리에 대해서 Pull Request 후 Merge를 이용한 저장을 허용해야 하므로 추가적인 설정이 필요합니다. 다시 레파지토리 Settings에서 Branch permission을 클릭합니다.

각 브랜치마다 퍼미션을 주면, 개발자는 바로 Push를 하지 못하고 Fork한 개인 레파지토리에서 PR를 진행해야만 메인 브랜치에 반영을 할 수 있습니다. 마스터 브랜치는 어드민만 권한을 가지게하고, development 브랜치는 Merge via pull request를 설정합니다. 연필 모양을 클릭하여 이것을 설정할 수 있습니다.

위와 같이 설정을 하면 최소 1명의 Reviewer가 승인을 하면 코드를 메인 레파지토리로 Merge할 수 있게 됩니다.

코드 리뷰를 하게 되면 리뷰어 및 개발자 모두에게 이득입니다. 리뷰어는 자신이 몰랐던 개발 방법이나 패턴을 익힐 수 있는 방법이 되고, 개발자는 인지하지 못한 비즈니스 로직을 이해할 수 있고 오류를 고칠 수 있어 깨끗한 코드가 유지될 수 있는 방법입니다.

Happy coding 하세요.