Skip to content

Files

Latest commit

ae6b5d4 · Oct 26, 2021

History

History
41 lines (26 loc) · 2.18 KB

pull_request_template.md

File metadata and controls

41 lines (26 loc) · 2.18 KB

PR 체크리스트

PR을 올렸다면 아래 사항은 반드시 지켜주세요.

  • PR 우측 Linked Issues에서 적절한 이슈 하나를 걸어두었습니다.
  • PR 우측 Labels에서 적절한 라벨을 부착하였습니다.
  • PR 우측 Projects에서 적절한 프로젝트를 할당하였습니다.
  • 본 PR을 팀원들과 사전에 상의하였고 팀원들 모두 해당 PR에 대하여 알고 있습니다.
  • 제가 의도한 파일들과 수정 사항들만 커밋이 된 것을 확인 하였습니다.
  • git rebase와 squash를 했고 commit이 하나인 것을 확인 했습니다.
  • 중요! 최초 PR 작성 후 수정 commit을 작성하신 경우에도 꼭 rebase해서 commit을 하나로 만들어줘야 합니다.

아래 사항까지 모두 체크되어야만 리뷰 요청 라벨을 달 수 있고, code review를 받을 수 있습니다.

  • 기능이 제대로 실행되는지 확인 하였습니다.
  • npm start로 구현한 화면 혹은 기능을 확인할 수 있습니다.
  • self-refactoring으로 wecode의 코드 스타일 가이드에 맞추어 코드를 작성 하였습니다.

아래 사항까지 모두 체크되어야만 merge가 될 수 있습니다.

  • 코드 리뷰 사항을 모두 반영하였습니다.

마크다운에서 체크를 하기 위해서는 [x] 이렇게 수정하시면 됩니다. [x ], [ x ] 가 아니라 정확하게 [x]여야 합니다. 혹은 마우스 클릭으로 체크해주세요.

구현 화면

(여기에 구현 화면 스크린샷을 첨부해주세요)

이슈 체크리스트

(여기에 이슈에서 계획한 상세 구현 사항들을 체크리스트로 만들어 구현할 때마다 체크해주세요.)

  • (예시) 회원가입 페이지 레이아웃 구현
  • (예시) 아이디, password validation 기능 구현
  • (예시) 이용약관 체크박스 기능 구현

기타 질문 및 특이 사항

(궁금한 사항들, 하면서 느낀 점들 공유해주실 것이 있으면 자유롭게 작성해 주세요.)

  • (예시) 이용약관 체크박스 기능 구현 로직이 너무 지저분하고 긴 것 같습니다. 좀 더 효율적인 방법이 없을지 궁금합니다!