-
Notifications
You must be signed in to change notification settings - Fork 0
PR Rule
suhyun_ edited this page Oct 9, 2024
·
1 revision
1. PR 이름 : Branch name
feature/1 feat 시작화면 개발
feature/1 feat 사용자ID 개발
2. 내용 : PR 템플릿 양식에 맞춰 작성한다.
### #️⃣ Related Issue
> ex) #이슈번호
### 📝 Work Detail
> 이번 PR에서 작업한 내용을 간략히 설명해주세요.(이미지 첨부 가능)
> 여러 작업을 진행할 경우, 작업이 끝나기 전 PR을 먼저 올리고 작업이 진행될 때마다 작업 내용을 업로드해주세요.
> Issue에서 작성한 Todo의 작업을 하나씩 수행할 때마다 PR에 그 내용을 수정/추가 해주세요.
### ScreenShots (선택)
### 💬 Review Requirements (선택)
> 리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.
>
> ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?
> merge하고 싶어요!
3. Reviewers : 리뷰를 원하는 개발자를 선택한다.
→ FE 개발자 : 본인 외의 FE 개발자(연동 후, BE 개발자)
→ BE 개발자 : 본인 외의 BE 개발자
4. Assignees : PR을 올리는 사람을 선택한다.(본인)
5. Labels : FE/BE + 관련 이슈 + 담당자 로 설정
- base branch는 develop-FE 또는 develop-BE로 해야한다.
- 가급적이면 FE/BE 개발자끼리 코드 리뷰를 한다.(연동할 때는 FE,BE 개발자가 함께 코드 리뷰를 한다.)
- 리뷰를 통해 merge허가를 받는다면, PR을 작성한 개발자는 merge한 후, PR을 close한다.
- closed된 branch는 꼭 삭제하고 Issue도 close한다.