Pull Request (PR) rule (브랜치 전략)
branch naming 전략
제한
Review
main
무조건 PR을 통해서만 push 가능하게 protection
dev
무조건 PR을 통해서만 push 가능하게 protection
feat/ksj-user (이름-도메인)
Merge는 1명이 approve했을 때 가능.
PR 시 구두로 코드 리뷰 후 approve 진행.
Approve guideline
필수
ERD설계와 API설계를 잘 적용하여 개발하였는가?
메서드와 변수는 가독성이 좋게 네이밍되었는가?
코딩 컨벤션을 준수하였는가?
예외 처리가 잘 되어 있는가?
권장
매직 넘버는 상수로 처리했는가?
else 사용을 지양하였는가?
각 메서드의 길이는 15줄을 넘지 않는가?
함수의 Parameter는 유효한 값을 가지고 있는가?
사용되는 변수들은 상황에 맞게 적절하게 선언되어 있는가?
변수들이 사용되기 전에 초기화되어 있는가?
코드가 무한 루프에 빠지는 상황은 없는가?
발견된 버그는 모두 올바르게 수정되었는가?