어떻게 커밋하지?
테스트 코드와 소스코드는 분리해서 커밋할지 고민이 있었고, 이 부분에 대해서도 커뮤니티에서 토론이 있었다.
*소스 코드 커밋
feat: 사용자가 입력한 자동차 이름의 유효성 검사 기능 추가
feat: 사용자가 입력한 시도 횟수의 유효성 검사 기능 추가
*테스트 코드 커밋
test: 자동차 이름과 시도 횟수 유효성 검사 테스트 코드 추가
커밋에도 관심사 분리
나는 코드 리뷰를 하는 사람의 입장을 고려해 분리해서 커밋하기로 결정했다. 보는 사람의 관심사에 다를 수 있기 때문이다.
예를 들어 CI/CD (Continuous Integration/Continuous Delivery) 배포 단계 중, CI 단계에서는 테스트 올렸을 때 통과되면 리뷰하는 사람한테는 관심이 없고 소스코드가 관심 있을 것이다. 반대로 만약 CI가 제대로 되지 않는다면, 그때 되어서 테스트 코드를 작성한 커밋으로 가서 테스트 코드를 보면 될 것이다.
즉, 보는 사람의 관심사를 고려해 시간을 아껴주는 차원으로 생각했다.