[Report] 사수에게 칭찬받는 PR체크리스트

주제

📌 사수에게 칭찬받는 PR체크리스트

Report

• Basic
  - 주석처리 남긴 것 지우기
  • 콘솔 로그로 확인한 것 지우기
  • 공통 컴포넌트 수정할 일 있으면, 다른 곳에 영향 없는지 확인하기
  • 커밋메시지 성심성의껏 남기기
• JavaScript
  - img path변수에 담을 때 끝에 / 붙이지 않고 사용할때 붙여 쓰기
  • 컴포넌트를 만들면 텍스트,이미지,크기는 최대한 표준이 될 수 있는 값으로 만들기
  • 변수명 성심성의껏..!
  • {}안 쓸수 있으면 쓰지 않기
  • if문은 중첩보다, 조건마다 return 시키는게 좋음!
• Styled-components
  - import시에 alias 적극 활용!
  • css가 없는 컴포넌트면 html태그로 수정
• TypeScript
  - 앵간하면 any는 쓰지 말자,,
  - interface나 type의 변수명,enum의 key와 value는 PascalCase로 작성!
  - interface 필수값 여부는 항상 체크하자. ?는 헷갈린다
  - tsx문법 사용여부에 따라 확장자명을 .ts || .tsx로 명확히 구분하자
  -React
  - Props는 immutable이다. 수정이 필요하면 복사해서 수정하자!
  - Props로 함수를 넘길때 parameter가 없으면 arrow function쓰지 말자 (함수 또 만드는 거임)
  - Props로 setState같은 함수를 직접 넘겨주지말고, handler함수를 만들어서 전달하자
  - 컴포넌트 재사용을 항상 고려하자 !

혼자 이해할 수 있는 코드를 짜는 이기적인 개발자가 되지 말자 !

출처 : 사수에게 칭찬받는 PR체크리스트