프론트엔드
코드 리뷰 in 뱅크샐러드 개발 문화
두줄요약
작은 PR과 실험 플랫폼으로 대규모 iOS 변경의 리뷰·병합 병목을 줄였습니다. PR 템플릿, 우선순위 규칙, Jenkins 테스트 자동화로 비동기 협업 품질을 높였습니다.
문제 상황
- 대형 PR로 인한 리뷰 집중도 저하와 일정 지연, 메인 브랜치 병목
- 대규모 UI/UX 개편과 정기 배포 병행 시 장기 브랜치의 노후화 및 Conflict 위험
- 텍스트 기반 비동기 리뷰에서 문맥 부족과 의견 강도 오해 가능성
해결 방법
- PR당 1,000라인 제한, 최소 기능·커밋 단위 분리, Mock JSON 중심 테스트 코드는 제한 예외
- 실험 플랫폼으로 동일한 메인 브랜치에서 실험군·대조군 기능을 격리하고 작은 PR 단위 병합
- PR 템플릿에 기획·기술 스펙·디자인·변경점·테스트 정보를 포함한 저문맥 리뷰 환경
구조와 흐름
- GitHub PR의 Approve·Comment·Request Changes 기반 비동기 코드 리뷰
- P1~P5로 수정 의견의 중요도 표시, D-0·D-N으로 리뷰 기한과 우선순위 공유
- SwiftFormat 빌드 단계 적용과 Jenkins 기반 PR 트리거 유닛 테스트·병합 차단 자동화
성능/운영 포인트
- 200~300줄 내외 PR과 1~2일 내 리뷰 완료로 리뷰 병목 완화
- 800여 커밋으로 대규모 변경을 세분화해 Conflict와 리뷰 지연 감소
- 별도 맥프로와 CI로 긴 iOS 유닛 테스트 실행 시간의 개발자 대기 비용 절감
