코드리뷰에 대해서

• 코드리뷰를 하는 이유
  다수 프로그래머가 코딩에 참여했을 때, 최선의 코드를 작성하기위해 코드리뷰를 진행합니다.

• 코드리뷰 받을 때
  코드 != 나
  나에 대한 평가가 아니라 그저 코드에 대한 리뷰
  리뷰를 해주는 것이 당연한 것은 아니다!
  리뷰어에 대한 감사를 느껴야합니다.

코드리뷰

프로그래머도 사람인지라, 코드리뷰를 하게되면 나에대한 평가로 느껴질 수 있습니다. 해당 Pn룰을 접목시켜서 코드리뷰를 진행하면 좋습니다.

• P1 : 이번에 반드시 반영되어야 하는 중대한 코드 수정 의견(Request Changes)
  버그 가능성이 있거나 잘못된 구현인 경우, 만약 반영되지 않는다면 이에 대한 반대 의견도 낼 수 있어야 합니다.
• P2 : 적극적으로 이야기했으면 하는 의견(Request Changes)
  잠재적인 이슈나 확장성을 고려해야 하는 경우. 토론하며 의견 조율할 수 있습니다.
• P3 : 가능하다면 반영해주었으면 하는 의견(Request Changes)
  지금 구현보다 더 나은 방향이 있는 경우. 이번 반영이 어렵다면 다음 작업에서도 고려해볼 수 있도록 합니다.
• P4 : 다음에 반영해도 되는 의견 (Approve)
  반영이 되지 않거나 반대 의견을 적극적으로 할 필요 없습니다.
• P5 : 사소한 의견(Approve)
  무시해도 됨. 혹은 관련 나누고 싶은 점 나눌 수 있다.

nit 줄이기

• 사소하고 작은 문제로 주로 스타일 가이드 준수, 가독성 개선 등을 의미
• 전체적으로는 중요하지 않지만 품질 향상에 도움이 될 수 있는 부분에서 사용

크롬 익스텐션

• WorkerB for Pull requests
• PR 화면에 대략적인 리뷰 시간 계산, slack 공유용 링크도 자동으로 생성 해줍니다.