-
Notifications
You must be signed in to change notification settings - Fork 4
커밋, PR, 코드리뷰 룰
yejineee edited this page Oct 29, 2020
·
5 revisions
- Feat: 새로운 기능을 추가할 경우.
- Fix: 버그를 고친 경우.
- Docs: 문서 수정한 경우.
- Style: 코드 포맷 변경, 세미 콜론 누락, 코드 수정이 없는 경우.
- Refactor: 프로덕션 코드 리팩터링.
- Test: 테스트 추가, 테스트 리팩터링 (프로덕션 코드 변경 없음).
- Chore: 빌드 테스크 업데이트, 패키지 매니저 설정할 경우 (프로덕션 코드 변경 없음).
- 본인 제외하고 최소 2명이 approve해야 merge할 수 있다.
- **퇴근시간 최대 21시❗️❗️❗️ === 9시까지만 PR을 보낼 수 있다.
- merge할 때 이슈를 관리한다.
- 1 issue === 1 pr (to master) (뱅크샐러드-하루에-1000번-배포하는-조직-되기)
-
이슈 처리를 완료하여 PR을 작성한다.
-
PR에는 모든 팀원을 리뷰어로 지정해야 한다.
-
슬랙에
@here로 PR 한 사실을 알리고, 팀원들에게 리뷰를 독촉한다.🤗 🤗 🤗 -
팀원은 코드를 보고 리뷰를 한다.🙆♂ 🙅♀
1️⃣ Week1
목표
데일리스크럼
회의
회고
2️⃣ Week2
목표
데일리스크럼
회의
회고
2️⃣ Week3
목표
데일리스크럼
회의
회고