PR 시 체크리스트 (지극히 개인적)

Cramming An·2022년 3월 30일
0
post-thumbnail

PR 시 체크리스트

인턴십을 진행하면 받은 PR 리뷰에서 받은 공통적인 피드백을 정리해봤습니다.

네이밍 문제

앞으로는 간단한 변수와 함수라도 처음 코드를 보는 사람입장에서 적고, 일관적이 있어야 할 것 같습니다.

반복적으로 사용되는 요소 모듈화 시키기

  • api 호출 주소와 같은 string 타입의 값을 바로 넣지 말고, 따로 모듈화를 해야 나중에 에러를 찾을 때 훨씬 수월할 것 같습니다.
  • 반복적으로 사용될 hook 또는 custom hook을 hooks 디렉토리에서 관리가 필요할 것 같습니다.
  • immutable 변수 만들도록 노력하기
profile
La Dolce Vita🥂

0개의 댓글