-
Notifications
You must be signed in to change notification settings - Fork 3
Ground Rule
Eunsol Lee edited this page Nov 2, 2020
·
3 revisions
- 팀 그라운드 룰
-
코드 컨벤션
- 네이밍 컨벤션 - 네이버
- eslint는 Airbnb 룰로 한다.
- 인덴트는 깊이를 2로 제한한다.
-
깃 커밋 컨벤션 :
[타입] 제목
- 기능(Feat): 새로운 기능을 추가
- 패키지(Package) : 패키지에 관련된 모든 변경
- 버그(Fix): 버그 수정
- 리팩토링(Refactor): 코드 리팩토링
- 형식(Style): 코드 형식, 정렬, 주석 등의 변경(동작에 영향을 주는 코드 변경 없음)
- 테스트(Test): 테스트 추가, 테스트 리팩토링(제품 코드 수정 없음, 테스트 코드에 관련된 모든 변경에 해당)
- 문서(Docs): 문서 수정(제품 코드 수정 없음)
- 기타(Chore): 빌드 업무 수정, 패키지 매니저 설정 등 위에 해당되지 않는 모든 변경(제품 코드 수정 없음)
-
PR 컨벤션
- task단위로 PR한다.
- PR 제목 : [안치현,이헌준] 로그인 추가
- PR 내용 : 참고사항, 설명(이해를 돕기 위한)
- PR은 feature브랜치로 날리고, 6시 이전에 개별적인 PR을 보내고 이후 작업은 자유롭게 진행한다.
- 다른 팀원의 PR에 대한 코드 리뷰는 7시까지 진행한다.
- 코드 리뷰를 통한 리팩토링은 다음 스크럼 이전까지 자유롭게 진행한다.
- PR Merge는 10시 30분 부터 다같이 진행한다.(스크럼 이후)
- PR에 모든 구성원을 리뷰어로 추가한다. 가능한 많이 리뷰시간을 갖는다.
- 모든 구성원이 approve해야 merge된다.
- 그 전에 PR을 한 사람은 조원을 돕자!
-
브랜치 관리 :
feature/내용
- [master] - [dev] - feature 브랜치들...
- "기획서 분석"의 기능단위로 feature 브랜치를 분리하여 task를 관리한다.
- 기능 구현을 마치면 feature 브랜치를 dev 브랜치로 Merge한다.
- dev 브랜치는 테스트를 마치면 master 브랜치로 Merge 후 배포한다.
-
조칙
- 1시~7시 에는 모두 줌에 들어와서 개발한다.
- 카메라는 꺼도 되지만 소리는 켜둔다.
- 지각은 없다.
- 리뷰를 청유형으로 하자.
- 막히는 부분에서 조금만 삽질하다가 물어보자.
- 세부적인 내용은 앞으로 정해 나가자.
- 1시~7시 에는 모두 줌에 들어와서 개발한다.
-
🔃 변경 이력
- 20.11.02 PR 방식 변경