Ground Rule

[yeonns]

Description

개발자는 project의 ground rule을 정한다.

Acceptance Criteria

• project 팀원에게 ground rule이 공유된다.

[yeonns]

Ground Rule

• PR은 feature단위로 생성한다 (최대 300줄 미만, 파일 10개 이내)
• 매주 수요일 PM 08:00에 진행상황을 공유하는 미팅을 진행한다
• PR description은 팀원을 위해 상세하게 작성한다
• PR은 최소 1명의 code review를 받아야 한다
• PR merge는 PR을 생성한 사람이 진행한다
• 기능 개발 시 test code가 같이 작성되어야 한다
• LGTM 리뷰는 지양하고, 서로를 존중하며 코드 리뷰가 이루어져야 한다
  • ex) 이 부분은 가독성을 위해 변수명을 XX로 변경하는 건 어떨까요?
  • ex) 이 line이 이해가 가지 않는데, 설명해주실 수 있을까요? 👀
  • ex) factory method pattern을 통해 구현하면 더 좋을 거 같습니다 😄 (예시 코드 OR 유용한 링크도 함께 공유필요)
• 코드 변경 제안을 리뷰어에게 강요하지 않는다.