• 기능 단위로 브랜치를 사용하시는 것을 추천드립니다. 버전까지 관리하기는 어려울 수 있으니 master <- develop <- feature branch 정도만 사용하셔도 큰 무리는 없을 것으로 생각됩니다. Git Flow 를 참조하시는 것도 좋습니다.
• 의미있는 최소 단위로 Commit하는 것을 추첫 드립니다.
• 커밋의 메세지는 AngularJS Git Commit Message Conventions) 을 권장 드립니다.
• Pull Request 하기 전 테스트 코드와, 테스트는 충분히 수행하였는지 항상 점검해 보시길 바랍니다.
• Pull Request는 최소 기능단위를 권장합니다.
• PR 메세지에는 다음과 같은 사항을 포함시켜 리뷰하는 사람들에게 설명 가능한 내용이 포함되어 있어야 합니다.
  • 변경점 혹은 구현된 기능과 그 설명
  • 구현된 기능에 대한 Test Code의 의도
  • Test code 외 실제 테스트도 진행하였다면 테스트 방법과 의도, 그 결과
• Code review는 여러분이 작성한 코드를 비난하기 위한 것이 아닙니다. Code review를 진행해 주는 사람에 대한 감사한 마음을 가집시다.
• Core review 를 진행하는 reviewer는 명령형이 아닌, 부드러운 말을 사용해야 합니다. 또한 무작정 이 부분이 잘못되었다! 라고 하기 보다는 이렇게 작성할 경우 이러이러한 문제점이 있을 것 같으니, 이런 구현 방법도 고려해 주셨으면 좋겠습니다. 정도의 부드러움이 필요합니다.
  • 뱅크 샐러드에서 사용하고 있는 Pn rule 도 참조하시면 좋을 것 같습니다.