뱅크샐러드의 코드 리뷰 문화
Source
Evernote/Inbox/코드 리뷰 in 뱅크샐러드 개발 문화 뱅크샐러드.md
Summary
뱅크샐러드는 비동기 커뮤니케이션과 GitHub PR을 기반으로 한 코드 리뷰 문화를 운영한다. 리뷰 병목 현상 해소와 조직 확장성을 위해 ‘작은 PR 규칙’(1PR 당 1,000 라인 이하, 테스트 코드 제외)을 도입하여 리뷰 효율을 높였다. 또한 데이터 기반 의사결정을 위해 실험 플랫폼을 활용하여 다수의 A/B 테스트를 유연하게 관리하며, 작은 PR과 실험 플랫폼의 조합을 통해 대규모 UI/UX 개편 프로젝트(뱅크샐러드 2.0) 등을 성공적으로 진행했다.
Key Points
- 비동기 커뮤니케이션 중심: GitHub PR을 통해 리뷰어의 명확한 피드백(Approve, Comment, Request Changes)과 병렬 작업을 지원하며 개인의 집중 시간을 존중한다.
- 작은 PR 규칙: 리뷰의 질 저하와 병목 현상을 방지하기 위해 1개 PR 당 코드 라인 수를 1,000 줄 이하로 제한한다. 테스트 코드는 라인 수 제한에서 제외된다.
- 실험 플랫폼 연동: 데이터 기반 의사결정을 위해 모바일 앱 내에서 다수의 실험(2020년 11월 기준 50개)을 동시에 운영하며, 실험군/대조군에 따른 UI/기능을 유연하게 제공한다.
- 시너지 효과: 작은 PR과 실험 플랫폼의 조합은 대규모 프로젝트(예: 뱅크샐러드 2.0)에서도 복잡도를 제어하고 지속적인 배포를 가능하게 한다.