forked from codesquad-members-2021/sidedish
-
Notifications
You must be signed in to change notification settings - Fork 2
0419 회의
Autumn (조홍비) edited this page Apr 19, 2021
·
5 revisions
텀: 문서화 잘 해보고 싶다. 에픽, 스토리, 태스크로 나눠보는 연습 해보고 싶다.
젠: 상태 관리 어렵.
델: 브랜치. develop에 머지하고, dev에서 PR 날렸음. dev 머지할 때 단순히 머지 ㄴㄴ 서로 리뷰해주자! (p1~p5 이용) 마무리톡
- 무시하지 않기!
- 화내지 않기!
- 답답해하지 않기!
- 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 등 이렇게 너무 일반적인 이름짓지 않기
- 텀: 파일이름은 소문자로 시작하는 카멜케이스, 컴포넌트는 대문자로 시작하는 카멜케이스로 썼어요. arrow function을 조와합니다. 저됴 jsx 대문자로 가욜~!
- 델: 저는 컴포넌트, 파일 모두 대문자로 시작하는 카멜케이스 햇숨다. 저도 애로우 애정함다.. 앗참 저도 컴포넌트는 jsx, 순수 js는 .js로 햇슴다 완찬~~~
- 젠: 아토믹한애들? 버튼,인풋이런애들은 그냥 Button.js, Input.js 다른 컴포넌트는 그냥 TodoList=> TodoListItems 카멜케이스로 한것같습니다..오홍...! 파일이랑 컴포넌트 다 대문자였던것같습니당. 저도 컴포넌트도 애로우로!