-
Notifications
You must be signed in to change notification settings - Fork 4
GitHub 규칙
Minjae Kim edited this page Nov 27, 2023
·
5 revisions
Status | 작업 설명 |
---|---|
No Status | 아직 할당되지 않은 작업 |
Todo | 할당이 정해진 할 작업 |
In Progress | 진행중인 작업 |
In Review | 작업이 끝나고 PR 리뷰를 기다리는 작업 |
Merged | Merge 완료된 작업 |
- 제목만 작성합니다.
- pr 단위와 동일합니다.
브랜치명 | 설명 |
---|---|
main | 배포를 위한 브랜치 |
develop | 개발을 위한 브랜치 |
feature | 이슈단위 기능 작성을 위한 브랜치 |
hotfix | 급한 이슈가 생겼을 시 문제를 해결하는 브랜치 |
refactor | 코드를 리팩토링하는 브랜치 |
feat/#이슈번호-기능명(길어지면 스네이크케이스)
ex) feat/#11-button-test
⚙️ PR 타입
- Feature
- Hotfix
✨ 기능 설명 or 🚨 문제 상황
👨💻 구현 내용 or 👍 해결 내용
🎯 PR 포인트
📝 참고 사항
❓ 궁금한 점
- label로 pr 우선순위를 구분합니다.
- 1명의 approve를 받아야 merge할 수 있습니다.
- 페이지 하나는 너무 크다.
-
300줄 미만으로 작성합니다.
- mock 데이터 제외
공통시스템개발팀 코드 리뷰 문화 개선 이야기 | 우아한형제들 기술블로그
- feature → develop : squash & merge
- hotfix → develop : squash & merge
- refactor → develop : rebase & merge
- develop → main : rebase & merge
- 긴급, 기능, 버그, 리팩토링
-
D-n 룰
-
긴급한 수정사항으로 바로 리뷰해 주세요. 앱의 오류로 인해 장애가 발생하거나, 빌드가 되지 않는 등 긴급 이슈가 발생할 때 사용합니다.
-
D-N (Within N days)
“Working Day 기준으로 N일 이내에 리뷰해 주세요”
-
feat: #1 xxx 기능 추가
fix: #2 xxx 기능 수정
docs: #3 xxx 문서 수정
style: #4 xxx 코드 수정
refactor: #5 xxx 기능 수정
chore: #6 xxx 버전 수정
design: #7 xxx 페이지 수정
1 commit 1 action을 지향합니다. 최대한 작은 단위로 커밋하려고 노력합시다.