목표
- 리팩토링 기간
- 자기가 맡았던 부분의 코드 퀄리티 향상
- 리뷰를 빡세게
기간
- 2주 + 4주 (~2022.09.20)
특이 사항
- 교체 작업은 날을 잡고 같이 하기
- 그 전까지는 리팩토링 작업을 진행
- 리뷰 하기 좋도록 300줄 미만으로 PR을 맞춰주세요
- commit 기준을 잘 잡는 연습도 같이 합시다
- 코드 퀄리티 향상을 위해 리뷰는 가감없이?
- axios instance를 하나 더 만들고, 자기가 맡았던 부분의 api를 점진적으로 교체
- 점진적으로는 어렵다.
- 교체하면서 동작되지 않는 부분을 기록 후 strapi에서 교체
리뷰 시간
- 다음 주 까지는 리팩토링이 어렵지만 리뷰는 가능
- PR 올라온 뒤 48시간 이내 리뷰 완료 (최소 2인 이상의 승인 후 머지)
- 2인 이상으로 변경
스케쥴 공유
- 일단 2주 경과한 뒤 스케쥴을 보고, api 교체 날짜를 정합시다
- 슬랙 스레드로 다시 미팅 날짜 정해서 빠른 미팅 ㄱㄱ
컨벤션
- 코드 컨벤션, 파일 컨벤션 ( + 폴더 구조…)
- api 로직 작성 부분이나, 인터페이스 정의 부분 등 자유롭게 적어주세요.
- 전 팀에서 컨벤션 어떻게 썼는지 공유 ( 팀 컨벤션 모음 노션에다가 붙여넣기)
- 댓글로 의견 공유
리팩토링 기준
- 직관적인 변수명
- 단일 책임 원칙
- 결합도는 낮추고, 응집도는 높인다
- 컴포넌트를 봤을 때 로직 / UI / 이벤트 3가지를 분리한다.
- 다른 사람이 작성한 코드를 봤을 때, 비슷하게 느껴져야 한다.
- 리팩토링은 성능 개선, 새로운 기능 추가의 전 단계
- 코드 컨벤션 & 파일 컨벤션 정하기
- 코드를 보고 느껴지는 문제점들 공유하기
- 각자 생각하는 리팩토링의 방향 + 패턴
9