Code Review
모두개발 배은경
modoo branch 전략
code review 유형
+ / -
결론
modoo branch 전략
Review 유형
- 컨벤션/네이밍
- 코딩스타일 논의
- 더 나은 로직 제안
- 재활용 가능한 코드 제안
- 스펙 문의 / 기존 스펙 확인
- 건설적인 토론
- 새로운 공부
- 미리 발견하는 버그
- ...
Case 1. 컨벤션/네이밍
Case 2. 코딩스타일 논의
Case 3. 더 나은 로직 제안
Case 4.
재활용 가능한
코드 제안
Case 5. 스펙 문의 / 기존 스펙 확인
Case 6. 건설적인 토론
Case 7. 새로운 공부
Case 7. 새로운 공부
Case 7. 새로운 공부
Case 7. 새로운 공부
Case 7. 새로운 공부
Case 8. 미리 발견하는 버그
Case 9. 미리 발견하는 버그
Case 9. 미리 발견하는 버그
- 업무 속도 저하
- dev 반영이 늦어짐에 따른 기획/QA 확인 속도 저하
- Context Switching 비용 발생
- 감정이 상할 수 있다
- 선머지 / 후리뷰
- 적절한 리뷰 단위 선택
- 리뷰는 작게, 자주
- 리뷰 시간을 따로 가져본다
- 하루 30분 정도, 아무리 길어도 1시간 넘지 않게
- 수정 강요는 금지, 단 버그는 필수 수정
- 부드러운 대화를 하기 위해 노력
코드 리뷰는 버그 잡기가 아니다
코드를 통해 서로 소통하고 배움을 주고 받는 것
좋은 코드의 공유
개발자의 지속적인 성장
팀 내 협업 강화
공감대 형성이 중요
겸손하고 정중하게 리뷰하기
리뷰 받을 코드라고 생각하면 좀 더 신경쓰게 된다
리뷰어도 자신의 지식을 점검하는 계기
Code Review
By realgo
Code Review
modoo case
