Skip to content

Latest commit

 

History

History
25 lines (21 loc) · 2 KB

codereview.md

File metadata and controls

25 lines (21 loc) · 2 KB
Raw

Keyword

code review

Reference

상황/궁금

  • 코드스쿼드의 '클린코드,리팩토링,TDD'강의에서 진행한 코드리뷰와
    • 코드리뷰를 받는 입장이기때문에 어떤 부분을 리뷰하는지를 포인트로 정리
  • SLiPP의 테스트기반 개발 스터디에서 진행하는 코드리뷰에서 배운 것을 정리
    • 매 스터디시간에 전시간 코드리뷰를 짧게 하면서, 현업에서 실제로 어떤 식으로 코드리뷰를 진행하고 있는지 도형형님이 틈틈이 코멘트를 하시는데,
    • 기본 매너(리뷰된 내 작성코드에 응답 코멘트를 다는 것, 코드리뷰에서 감정을 건드리지 않도록 조심하자 등)와 현업에서의 적용(merging에 대한 의견)을 배울 수 있어서 유익하다
    • 시니어가 진행하는 코드리뷰를 엿볼 수 있는 거의 유일한 기회라서 도움이 된다.

정리 / 느낀 점

코드리뷰 기본

  • 코드리뷰에서 감정을 건드리지 않도록 조심하자
  • 리뷰된 내 작성코드에 응답 코멘트를 달아서 피드백을 주자.
    • 리뷰된 코드에 반영했습니다 등올 다시 코멘트 다는 거 잊지말자!
  • (팀마다 방식다름) merging시에도 다른 팀원들도 'ohahohah merge 찬성합니다' 등의 피드백을 주고 이를 확인 후 merge함
    • 보통 팀원들이 conversation에 자신이 코드리뷰했음을 코멘트함. merge에 대한 의견도 코멘트함.
  • PR 명세 수준 : '설계서 + 새로 작성한 명세서' 를 보고 다음 작업자가 이해할 수 있을 정도

에러발생시 알려주기