-
Notifications
You must be signed in to change notification settings - Fork 3
PR 컨벤션, PR 단위
DoDoBest edited this page Nov 7, 2022
·
1 revision
[PR] 간략한 작업 내용
PR Description
- 관련 이슈 : 개발하다가 발생했던 이슈 중에 공유하고 싶은 것
- 관련 문서 : 에러나 기능 구현시 참고한 자료
Changes
- 작업 내용
Screenshots
- 작업 관련 사진
유저 Story 1개 단위로 작업 후 올리기
- 한 사람의 PR을 2명이 코드 리뷰하기
- 리뷰는 최소 24시간 안에 달아주기
- 받은 리뷰에 대해 질문이 있으면 코멘트로 달기
- 받은 리뷰를 적용한 commit을 추가 코멘트로 달기
- 두 명이 해당 PR에 대해 괜찮다는 코멘트를 달면 본인이 Merge
본인 번호 아래 있는 두 사람 + 매주 한 명씩 아래로
ex)
1주차에 K008에 대한 리뷰는 K037, K039
2주차에는 K039, K048
3주차에는 K048, K037