-
Notifications
You must be signed in to change notification settings - Fork 4
개발 방식
Mirim Park edited this page Nov 11, 2022
·
5 revisions
- git-flow 전략
- 개발자용 서버 만들기
- PR은 본인 제외 2인 이상 리뷰해야 merge 가능 → 깃 마스터가 관리
- 각자 feature를 생성
-
git-cz (commitizen) 사용
-
Type
-
test
— Adding missing tests -
feat
— A new feature -
fix
— A bug fix -
chore
— Build process or auxiliary tool changes -
docs
— Documentation only changes -
refactor
— A code change that neither fixes a bug or adds a feature -
style
— Markup, white-space, formatting, missing semi-colons... -
ci
— CI related changes -
perf
— A code change that improves performance
-
-
커밋 메시지는 한글로 적기
- Typescript any 금지 (피치 못할 경우 주석 달아놓기)
Google TypeScript Style Guide - eslint, prettier
- 변수명, 함수명 규칙
- asset : 소문자 + -
- 리액트 컴포넌트, 클래스 : 파스칼
- 상수 : 스네이크케이스 + 대문자
- 함수 : 카멜케이스?
- 이벤트 핸들러 : 아래 참고
[번역] React의 이벤트 핸들러 네이밍 (Event handler naming in react)
- 주에 FE / BE 하나씩 로테이션
- 본인 텀이 아니어도 쓸 내용이 있으면 추가로 써도 됨
- 주말에 해도 됨
<Issue 템플릿>
## 이슈 내용
무슨 이슈인지에 대한 설명을 적어주세요.
## 기대 결과
- 원하는 결과물에 대한 내용을 적어주세요.
이슈 해결과정은 댓글로 남긴다.
<PR 템플릿>
## 개요
여기에 기능에 대한 간략한 설명을 적어주세요.
## 작업사항
- 여기에 기능에 대한 작업 사항을 적어주세요
## 리뷰 요청사항
- N/A
- PR Merge 승인 방식
Auto Assign Action을 이용한 2명에게 랜덤으로 리뷰어 할당 해당 리뷰어가 코드 리뷰 후 merge 승인
- PR 리뷰 마감일
19:00 이전의 PR은 다음날 데일리 스크럼 전까지 리뷰 완료 19:00 이후의 PR은 다음날 19:00전까지 리뷰 완료
- 📒 설계서
- 📕 기획서&디자인 피그마
- 🧾 백로그
- 🗂 API