새로 입사한 개발자가 프로젝트에 기여하는 방법 한 가지
이종립 컬리 · 컬리 기술 블로그
AI 핵심 요약
입사 초기 팀의 코딩 스타일 가이드를 살펴본 뒤, 중괄호 생략 등 컨벤션 위반을 찾아 작은 PR로 수정하며 프로젝트에 기여했다. CheckStyle을 프로젝트 전체에 적용하자 경고가 1,823개 발견됐지만, 포매터로 한 번에 바꾸는 대신 `ag`, `sed`, `find`를 활용해 영향이 적은 스타일 문제부터 나눠 고쳤다. 각 변경은 diff와 테스트로 확인한 뒤 PR로 제출해 코드 리뷰와 Git 활용을 익히는 기회로도 삼았다. 이런 점진적 정리로 경고를 500개 이상 줄였고, 새 프로젝트에서는 경고를 거의 없는 상태로 유지하는 한편 컨벤션과 PR 중심의 협업 문화를 다지는 데 기여했다.
- CheckStyle 경고를 한꺼번에 포매터로 수정하기보다, 영향이 적고 검증하기 쉬운 규칙부터 골라 처리한다.
- `ag`, `sed`, `find` 같은 터미널 도구를 조합하면 반복적인 스타일 위반을 찾아 일괄 수정할 수 있다.
- 자동 수정 뒤에는 `git diff`로 변경 범위를 확인하고 테스트를 실행해 의도치 않은 변경을 점검한다.
- 작은 수정도 PR로 나누면 코드 리뷰와 Git 사용법을 익히는 학습 자료가 될 수 있다.
- 보이스카웃 규칙처럼 발견한 경고를 조금씩 제거하면 코드 품질을 점진적으로 개선할 수 있다.
소개
프로젝트 전체의 경고를 감소 시켜 팀에 기여하자!
비슷한 학습 자료
문서 김경윤
AI가 팀 규칙을 지키도록 하는 방법
문서 이규현
우당탕탕 beauty 풀한, 컬리 앱 서비스 런칭기
문서 홍혜린
코드 악취를 맡는 후각 훈련의 시간
문서 한경훈