-
Notifications
You must be signed in to change notification settings - Fork 0
✅ Attention, please
Min_ehfdl edited this page Aug 9, 2022
·
5 revisions
- Code Convention은 HackMD 혹은 Wiki 참고
- 브랜치명은 ‘/‘ 뒤에 [이슈번호]-[영어로 어떤 story인지 표현(띄어쓰기는 -로 통일)] (ex. feature/12-Refactoring-Calendar)
- Xcode Project 내에서 MVVM 관련 파일 생성 시 [blahblahViewModel] 처럼 파일 이름 짓기 (혼선 방지용)
- Model 파일의 경우에만 이름 뒤에 Model 안 붙이기
- 이슈는 백로그의 Story 단위로 만들기
- 이슈 만들 때 프로젝트 선택해두기
- 이슈 만들 때 assignee 본인 설정
- 이슈 만들 때 label 설정
- PR은 task 단위로 올리기
- PR 요청사항 반영 시 하나씩 수정 후 이에 대한 커밋 남긴 뒤 커밋 주소와 같이 코멘트 남기기
- PR Merge 이후 더 이상 필요가 없다면 브랜치 삭제하기
- PR 올렸을 때 PR 올렸다고 단톡방에 언급하기
- PR을 올렸는데 하루 지나도 아무도 Approve를 안 해주면 단톡방에서 한명 지목해서 코드리뷰 요청하기
- 코드리뷰 남길 때 p1~p3 로 우선순위 판단할 수 있게 남겨주기 (칭찬은 매너⭐️)
- 코드에 관해서 질문 자유롭게 적어주기
- SendGift: 각각의 선물 보낼 때 View
- Gift: 각각의 선물 보관함 (받은 선물 보는 View)