우리의 영화가 시작되는 곳, CGV 🎬
35기 AND SOPT 합동세미나 앱 2팀 CGV 프로젝트입니다.
| 🔖 김수연 @mmaybei |
🔖 최안용 @ChoiAnYong |
🔖 김송희 @hongseekim |
|---|---|---|
 |
 |
 |
좌석 선택 뷰 |
시간 선택 뷰 |
홈 뷰 |
| library | description | version |
|---|---|---|
| Snapkit | UI 구현에 있어, 오토레이아웃을 보다 간편하게 사용하기 위함 | 5.7.1 |
| Then | UI 구현에 있어, 클로저를 통해 인스턴스를 초기화하기 위함 | 3.0.0 |
| Moya | 추상화된 네트워크 레이어를 보다 간편하게 사용하기 위함 | |
| Kingfisher | 이미지 캐싱 처리 및 UI 성능 개선을 위함 |
스타일쉐어 Swift 가이드를 기반으로 합니다.
한 줄의 코드가 100자를 넘어가는 경우, 파라미터 이름을 기준으로 개행합니다.
일반적이지 않은 함수나 변수의 경우 /// 문서화 주석을 사용하여 간단한 설명 남깁니다.
[init] 가장 처음 Initial Commit에 태그 붙이기!
[feat] 새로운 기능 구현 시 사용
[fix] 버그나 오류 해결 시 사용
[docs] README, 템플릿 등 프로젝트 내 문서 수정 시 사용
[setting] 프로젝트 관련 설정 변경 시 사용
[add] 사진 등 에셋이나 라이브러리 추가 시 사용
[refactor] 기존 코드를 리팩토링하거나 수정할 시 사용
[chore] 별로 중요한 수정이 아닐 시 사용
브랜치의 경우 이슈 번호를 붙여 다음과 같이 작성합니다.
setting/#1-initialSetting
feat/#14-homeUI
커밋의 경우 이슈 번호를 붙여 다음과 같이 작성합니다.
[setting/#1] 프로젝트 초기 세팅
[feat/#14] 홈 화면 UI 구현
develop 브랜치에서 이슈 번호에 맞는 feature 브랜치를 파서 작업을 진행하고, PR 또한 develop 브랜치로 날려줍니다!
PR을 작성하면 작성자 이외의 다른 팀원들이 코드 리뷰를 진행하고, 문제가 없을 경우 approve 합니다.
수정 사항이 있을 경우 반드시 수정 사항 반영을 마친 후 approve하도록 합니다. 이렇게 모든 팀원의 approve가 완료되면 PR을 머지합니다.
이때 충돌이 생긴다면 반드시 리드 및 팀원에게 노티하도록 합니다!
자신의 로컬 develop 브랜치에서 원격 develop 브랜치를 자주 pull 받음으로써 충돌 상황을 방지합니다.
이슈 및 PR은 작은 단위로, 상대방의 파일은 최대한 건드리지 않습니다.
건강한 코드리뷰 문화를 지향합니다.
(1) 그냥 제 의견은 그렇다구요~
(2) 이렇게 하면 더 좋을 것 같은데요?
(3) 이건 고쳐주세요!
(Q) 질문
코드 리뷰 시 중요도를 표기하여 생산성을 높입니다.
이때 (3)의 경우 반드시 반영한 후 머지하도록 합니다.