셀프리뷰 헤보기

[moonkii]

Merge 하시기 전에 셀프리뷰는 해보시나요?

지금은 혼자 작업 하시기 때문에 PR 을 보낸 뒤 바로 merge 해버리면 사실상 PR을 보내는 의미가 크게 없긴해요.
셀프 리뷰를 해보시고 만약에 수정해야할 사항을 발견하면 스스로 코멘트를 달고 그다음 수정하는 연습을 한 번 해보세요.
(커밋 메시지도 수정하실 때 그냥 Edit 하지 말고, 어떤 부분을 개선했다나 왜 그렇게 수정했는지 간단히 남겨봐도 좋을 것 같아요.)
코드 리팩터링에도 도움이 되고, 코드를 리뷰할 때 어떤 사고의 흐름으로 작업을 하셨는지도 리뷰어가 파악하는데 도움이 될 것 같아요!

[canary4651]

앗...! 급하다고 생각해서 놓치고 있던 부분이었어요.
셀프 리뷰 안하고 그냥 머지 하니까 다음 작업 할 때 고치는 게 많아져서 엄청 단위가 늘더라구요...
pr 보내는 횟수도 줄고 한번 할 때 뭉텅이로 올리지는 참사가...! 정신이 없네요 진짜 😂
바쁠수록 기본부터 더 잘 챙기도록 노력하는 다음 한 주를 보내겠습니다.