- 코드!=나
- 나에 대한 평가가 아니라 그저 코드에 대한 리뷰
- 리뷰해주는건 당연한 것은 아니다!
- 리뷰어에 대한 감사
코드리뷰
- 빠르게 리뷰하기
- P1 : 이번에 반드시 반영되어야 하는 중대한 코드 수정 의견(Request Changes)
- 버그 가능성이 있거나 잘못된 구현인 경우, 만약 반영되지 않는다면 이에 대한 반대 의견도 낼 수 있어야 한다.
- P2 : 적극적으로 이야기했으면 하는 의견(Request Changes)
- 잠재적인 이슈나 확장성을 고려해야 하는 경우. 토론하며 의견 조율할 수 있다.
- P3 : 가능하다면 반영해주었으면 하는 의견(Request Changes)
- 지금 구현보다 더 나은 방향이 있는 경우. 이번 반영이 얼벼다면 다음 작업에서도 고려해볼 수 있도록 한다.
- P4 : 다음에 반영해도 되는 의견 (Approve)
- 반영이 되지 않거나 반대 의견을 적극적으로 할 필요 없다.
- P5 : 사소한 의견(Approve)
- 무시해도 됨. 혹은 관련 나누고 싶은 점 나눌 수 있다.
- nit 줄이기
- 사소하고 작은 문제로 주로 스타일 가이드 준수, 가독성 개선 등을 의미
- 전체적으로는 중요하지 않지만 품질 햐상에 도움이 될 수 있는 부분에서 사용
- 변경 사이즈 줄이기
크롬 익스텐션
- WorkerB for Pull requests