코드 리뷰하기
리뷰를 받는 입장. Assignee
마음가짐
- 열린 마음: 리뷰어의 지적에 무조건 방어적인 태도로 받지 않기
- 열린 마음이되, 비판적인 사고를 유지하기
- 리뷰어의 리뷰 타당성에 대해 냉정하게 고민하기
리뷰하기 쉽게 만들기
- 리뷰 단위를 작게. -> PR or diff를 작은 단위로 쪼개서
- 리뷰해야 하는 점에 더욱 Focus
- (당연히) 코드를 간결하게
- 테스트 플랜을 쓰기
- Unit test 혹은 적어도 코드가 동작한 결과물(스크린샷 이라던지)
- 결국 코드의 신빙성을 높일 수 있다.
리뷰를 하는 입장. Reviewer
- 받는 사람의 입장에서 Soft하게 느껴질 수 있도록
- “제 생각에는… “을 입에 달고 리뷰하기
- 받는 사람이 빠르게 받아들일수록 팀 전체의 생산성이 증대된다.
- 정확하고 자세하고 actionable(뭘 해야하는지 명확하게)
- 코드 변경을 요청하기 전에 변경에 드는 노력과 노력대비 Return이 어느정도 될지를 생각하기
- 코드 push가 늦어질만한(생산성이 줄어도 될만한) 가치가 있는 피드백인지?