forked from codesquad-members-2021/sidedish
-
Notifications
You must be signed in to change notification settings - Fork 2
AJA 협업 룰
Autumn (조홍비) edited this page Apr 19, 2021
·
1 revision
- 무시하지 않기!
- 화내지 않기!
- 답답해하지 않기!
- 7시에 마무리톡 하기!
- PR에 넣고 싶은 메시지를 활동기록(Wiki)에 쓰기!
- PR은 최대한 자주! 월수금 계획!
- 구조를 바꾸고 싶으면 먼저 상의하기!
- 아자아자!
- 텀(📌텅 아님)장님 말씀 잘듣기!
- 10 to 7, PR 메시지를 자세하게 쓰자.
- PR은 12시에 보낸다. 슬랙으로 보내거나 위키에 활동기록(그날 한 일, 구현한 것 등) 정리해놓은 것을 바탕으로 보낸다.
- PR 후 fetch, rebase 한다.
- branch 만들 때 AJA > dev > feature (feature 브랜치 이름 - feature/XXX)
- PR은 dev > upstream으로, fetch, rebase는 upstream > AJA로
- feature > dev PR은 7시 마무리톡을 하면서 한다.
- 저녁 시간을 이용해서 각자 리뷰해주고 12시에 merge 후 upstream으로 PR
- upstream merge 전까지 dev는 건드리지 않기! (dev에서 태어난 prac 브랜치에서 연습하되 함부로 dev에 merge하지는 말 것)
[Add]: 페이지(html), 함수(js)가 새로 추가됨
[Fix]: 버그를 고침
[Modify]: 함수, 기능 등을 수정함
[Refactor]: 페이지(html), 함수(js) 가 필요없어서 삭제하거나, 이름이 마음에 안들어서 바꿈
[Style]: 스타일(css)이 추가, 수정, 삭제되거나 html 이랑 link 됨
[Docs]: 문서(md)가 추가, 수정, 삭제됨
[Test]: 테스트 코드를 추가, 수정 삭제함
[Chore]: 그외 기타 (오타 등 자잘한 수정 포함)
[Feat]: 기능이 완성됐을 때
- 제안을 할 때에는 근거를 함께 말하기
- 제안에 맞춰 p1 ~ p5 달아주기
리뷰어가 코멘트를 강조하는 정도를 표현하는 룰
P1 : 꼭 반영해주세요 (Request changes)
P2 : 적극적으로 고려해주세요 (Request changes)
P3 : 웬만하면 반영해주세요 (Comment)
P4 : 반영해도 좋고 넘어가도 좋습니다 (Approve)
P5 : 그냥 사소한 의견입니다. (Approve)
- 설계하면서 차차 하기로!
- 파일, 컴포넌트 이름: 대문자로 시작하는 camel case (파스칼 케이스)
- 컴포넌트는 arrow function으로 한다.
- data, name 등 이렇게 너무 일반적인 이름짓지 않기