4. 개발 후
-
PR 올리기 전 develop 변경 사항 있을 시 pull 받은 후 PR 올리기
-
PR 제목 피처명으로 통일(
_는 띄어쓰기로 대체)
-
머지하고자하는 브랜치로 지정되어 있는 지 확인 (default :
develop)
 |
- reviewers는 스프린트 회의 때 정한 리뷰어를 달아주면 됩니다. (3명 이상 approve 시 merge 가능합니다.)- assignees에는 pr에 관여한 사람 넣어주세요!(보통 본인을 지정하면 됩니다)- labels PR에 관련되 라벨 지정해주세요- projects로 KEEPER FRONT R2 달아주세요 |
|---|
 |
저희는 PR 템플릿을 사용하고 있습니다. PR 템플릿 양식에 따라 내용을 기입하면 됩니다. +)공통 컴포넌트 작업에 대한 PR은 코드 사용 예제도 추가해주세요! 이슈 제목을 포함한 멘션으로 이슈를 달기 위해 이슈 번호 앞에 - 을 달아주세요   해당 PR으로 작업이 끝나는 이슈일 경우 - Close #130 이렇게 close를 앞에 붙이면 PR 머지 시 이슈가 자동으로 close 됩니다. |
|---|
-
Re-request review 클릭하면 리뷰어에게 다시 알림이 가게 됩니다. 리뷰 피드백 반영후 재리뷰 요청할 때 눌러주세요.
-
머지는 특별한 경우가 아니면 본인이 합니다.
- 리뷰어는 24시간이 지나기 전 리뷰를 답니다.
- 리뷰이는 리뷰어의 피드백을 받고 피드백에 대한 답변을 24시간이 지나기 전에 달아줍니다.
[ 리뷰 관련 라벨 ]
-
Priority-High: 급하게 머지해달라 요구할 때 쓰는 라벨
-
Priority-Low: 리뷰 피드백 반영해서 수정 중이라서 추가적으로 확인 안해봐도 되거나, 급하지 않을 경우 쓰는 라벨
-
ex.
리뷰 어떻게 달아야 할 지 모르겠다!하면 아래 글 읽어보는 거 추천드립니다.
위의 글과 비슷한 내용이 포함되어 있지만 읽어보면 좋을 만한 글도 남깁니다.
리뷰를 스포츠에 비유한 글인데 실력이 부족한 사람들에 대해서도 리뷰를 이런 식으로 할 수 있다는 등 여러 좋은 내용이 많이 담겨 있는 것 같아 남겨둡니다!
저희는 PR을 사용하기 때문에 기능 마무리는 사용하기 어려울 겁니다.
따라서 PR이 머지되고, 해당 피처의 역할을 다했으면 직접 브랜치 로컬과, 원격에서 삭제 부탁드립니다.