-
Notifications
You must be signed in to change notification settings - Fork 3
2021.04.01 회의록
YONGJAE LEE edited this page Apr 1, 2021
·
3 revisions
-
npm vs yarn
- yarn 으로 변경
-
fork 받은 브랜치에서 PR시 다른 origin인 경우 해당 PR에 push
- TODO: PR에 해당하는 branch를 pull 받아서 push 할 수 있음(테스트 안해봄)
- https://blog.outsider.ne.kr/1204
-
주석 컨벤션
-
로직의 복잡성에 대한 내용
- 코드의 의도 => 특이한 것들 왜 그렇게 짯는가?
- 함수의 파라미터와 return 타입 정도만 정의되어도 수월하다
- TypeScript로 전환되면 괜찮아질 부분
-
extension
-
-
Immutable
- 기존 작업물 거의 적용 안되어있음 수정이 필요
- 앞으로 작업할 때 지켜서 하면 좋을 것 같음
-
Input
-
Test
- TDD로 진행하진 않음
- E2E 우선순위를 높게
- 유닛테스트는 커버리지에 연연하지 않게끔
- 명확한 것 말고 핵심 로직 및 중요하거나 의존성이 있는 부분을 테스팅
-
TypeScript
- TODO: js와 ts 병행 가능하도록 tsconfig 설정
-
postcss
- 보류
- TODO: styled 에서 이미 적용되는지 찾아야 함
-
Tailwind css?
- 보류
- TODO: styeld component와 병행하는게 일반적인지 찾아야 함
-
polished?
-
회의
- 스크럼: 뭘 했다 / 뭘 할꺼다
- 이슈 공유
-
모르는 부분, 이슈 있으면 저녁 6시 이후 요청 <- discord 음성채널에서 논의
-
쉬운 이슈 하나씩 잡아서 PR
- yarn 전환
- better comment config 추가