-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #440 from GSM-MSG/update-pull-request-template
PR 템플릿 업데이트
- Loading branch information
Showing
1 changed file
with
30 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,31 @@ | ||
## PR 정보 | ||
내용을 적어주세요. | ||
## 💡 배경 및 개요 | ||
|
||
## 작업 결과 | ||
- 내용을 적어주세요. | ||
> PR을 하게 된 문제상황, 배경 및 개요에 대해서 작성해주세요! | ||
> | ||
> 퍼블리싱의 경우 스크린샷/동영상도 추가해주면 좋아요! | ||
Resolves: #{이슈번호} | ||
|
||
## 📃 작업내용 | ||
|
||
> PR에서 한 작업을 작성해주세요! | ||
## 🙋♂️ 리뷰노트 | ||
|
||
> 구현 시에 고민이었던 점들 혹은 특정 부분에 대한 의도가 있었다면 PR 리뷰의 이해를 돕기 위해 서술해주세요! | ||
> | ||
> 또한 리뷰어에게 특정 부분에 대한 집중 혹은 코멘트 혹은 질문을 요청하는 경우에 작성하면 좋아요! | ||
> | ||
> e.g. 작업을 끝내야할 시간이 얼마 없어 확장성보다는 동작을 위주로 만들었어요! 감안하고 리뷰해주세요! | ||
## ✅ PR 체크리스트 | ||
|
||
> 템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요! | ||
- [ ] 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. `.env`, `노션`, `README`) | ||
- [ ] 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. `"API 개발 완료됐어요"`, `"환경값 추가되었어요"`) | ||
- [ ] 작업한 코드가 정상적으로 동작하나요? | ||
- [ ] Merge 대상 브랜치가 올바른가요? | ||
- [ ] PR과 관련 없는 작업이 있지는 않나요? | ||
|
||
## 🎸 기타 |