뱅크샐러드, 코드 리뷰 문화를 통해 개발 생산성을 높이다!
by DD
5년 전
조회수 11
코드 리뷰를 통해 기술 공유, 아키텍처 일관성 유지, 잠재적 버그 발견
작은 PR 규칙과 실험 플랫폼을 결합하여 뱅크샐러드 2.0 UI/UX 개편 성공
Pn/D-n 룰 및 유닛 테스트 자동화를 통해 개발 생산성 및 코드 품질 향상
비동기 커뮤니케이션과 코드 리뷰
뱅크샐러드는 비동기 커뮤니케이션을 지향하며, GitHub Pull Request를 코드 리뷰 도구로 활용한다. 따라서, Jira와 Slack을 통해 업무를 진행하고, 개인의 업무 시간을 존중한다. 구체적으로, PR을 통해 리뷰어의 의견을 명확히 표현하고, merge 여부를 결정한다.
작은 PR과 실험 플랫폼의 시너지
뱅크샐러드는 작은 PR 규칙을 통해 코드 리뷰 병목 현상을 해소하고, 실험 플랫폼을 통해 유연한 모바일 서비스 출시를 지원한다. 반면, 작은 PR은 기능 개발의 세분화를 요구하며, 실험 플랫폼은 다양한 실험 환경을 관리해야 한다. 결과적으로, 뱅크샐러드 2.0 프로젝트에서 800개의 커밋으로 성공적인 UI/UX 개편을 달성했다.
Pn/D-n 룰과 유닛 테스트 자동화
뱅크샐러드는 Pn 룰을 통해 코드 리뷰 코멘트의 중요도를 명확히 하고, D-n 룰로 리뷰 우선순위를 결정한다. 따라서, SwiftFormat을 도입하여 코딩 스타일 자동화를 구현했다. 결과적으로, 유닛 테스트 자동화를 통해 개발 생산성을 향상시키고, 테스트 시간 133시간 절감 효과를 얻었다.
