기본 규칙
❗️ 커밋은 가능하면 자주 할 것
❗️ 반드시 검토할 사람이 있을 때 PR을 올리고 slack에 공유하기
커밋 방법
•
[커밋 타입/#이슈번호] - 커밋 내용 형태로 작성
(ex. [feat/#12] - Router에 MainPage 연결 )
Github Type
작업 타입 | 작업내용 |
 [feat] | 초기 세팅, 새 파일 및 코드, 새 기능 |
| [fix] | 버그 및 기능 수정 |
| [refactor] | 코드 수정 |
| [del] | 파일, 코드, 기능 삭제했을 때 |
| [style] | CSS 를 바꿨을 때 |
| [test] | 더미데이터, 기능 테스트 |
| [docs] | 문서 수정 ex) readme, script package.json 변경(npm 설치 등) |
| [merge] | PR이 승인되고 머지 후 pull 받을 때 |
코드 시작 전 브랜치명
•
브랜치 이름 : 커밋규칙_기능/#이슈번호
◦
ex) feat_login/#1
이슈 생성
•
[타입] - 하려는 내용 할 내용을 제목으로 입력
•
Assignees 를 클릭하여 담당자(자기 자신) 지정
Issue Feature
전체적인 작업사항을 입력해주세요Todo작업사항
작업사항
이슈 서두 : 커밋 규칙을 적용
•
[feat] - 내용
이슈 본문
PR 규칙
•
PR 주의사항
◦
(중요) 반드시 검토할 사람이 있을 때 PR을 올릴 것!!
◦
새로운 라이브러리, 패키지 설치 시
◦
공통적인 부분 (ex. Route, 공통 컴포넌트, 공통 스타일 등)을 수정 시
•
PR (Pull Request) 수행 이후에 할 일
1.
Slack에 PR URL 주소를 올림
2.
다른 사람들이 PR을 검토하고 “승인” 하면 Merge 진행
3.
Slack에 Merge 했다고 메시지를 남김
•
Merge 수행 이후에 할 일
1.
팀원이 본인 브랜치에서 작업하던 지점까지 add + commit
2.
git pull origin dev
3.
추가된 라이브러리가 있는 경우 yarn
4.
yarn start 하여 충돌 여부 확인
5.
충돌 여부 Slack에 남기기
- 정상 작동 시 "확인했다" , "이상 없다"
- 충돌이 있을 시 "어떤 부분에서 충돌이 있다" , "충돌이 있었는데 이렇게 해결했다"