TIL
Chapter 16 코드 정리 구분
PR(Pull Request) 검토 모델을 사용한다고 가정했을 때 코드 정리는 어디서 해야할까?
별로 좋지 못한 모습의 사례
동작 변경 코드와 함께 코드 정리 내용 넣기
검토하던 사람들이 내 PR이 너무 길다고 불평
코드 정리 내용을 분리해서 동작 변경 PR 앞, 뒤에 둠
검토하는 사람들이 코드 정리만 담긴 PR을 이해할 수 없다며 불평
즉 코드 정리는 가급적 별도의 PR로 만들고 몇 개의 코드 정리만 넣는게 좋다.
변경을 구분하지 않은 상태에서 다수의 변경을 하면 안 된다
ex) if 문 변경 도중 메서드 이름을 변경하고 다시 if 문으로 돌아감
변경은 변경을 낳는다.
변경 대상엔 두 가지가 있다.
프로그램의 동작 변경 (프로그램 실행으로 찾는 것이 가능)
프로그램의 구조 변경 (코드를 자세히 살펴봐야만 관찰 가능)
동작 변경과 구조 변경은 한 PR에서 별도로 나타나면 좋다.
코드 정리와 동작 변경 사이를 번갈아 가며 전환할 때마다 새 PR을 열어야 한다.
작은 PR은 보통 검토 시간 단축으로 환영 받는다.
코드 정리에 익숙해져 더 작은 단위로 작업하고 절대적으로 안전하게 작업하는데 익숙해지면 용기를 내서 코드 정리 PR은 따로 검토하지 않는 시도를 해 보자.
대기 시간이 더 줄어들어 더 작은 PR을 만들도록 장려하는 효과가 생길 것이다.