LDD팀 신규 입사자, 터미널 도구 활용해 코드 컨벤션 위반 500건 이상 제거
·2020.03.09 16:20
핵심 내용
LDD팀 신규 입사자가 터미널 명령어로 코드 컨벤션 위반 500건 이상을 제거하고 팀 내 코드 리뷰 문화를 정착시켰다.
자세히 보기
LDD팀의 신규 입사자 이종립은 입사 첫 주 동안 팀의 코딩 스타일 가이드를 준수하기 위해 터미널 도구를 활용한 자동화 작업을 수행했다. 그는 find, egrep, ag, sed 등 명령어를 조합해 중괄호 생략, 공백 누락, 탭 문자 사용 등 500건 이상의 컨벤션 위반 사례를 찾아내 수정했다.
점진적 개선 전략
전체 포매터 일괄 적용 대신 **보이스카웃 규칙(Boy Scout Rule)**을 적용해 발견할 때마다 조금씩 청소하는 방식을 선택했다. 이는 신규 입사자가 Git, 터미널 명령어, Pull Request 워크플로우를 학습할 수 있는 기회로 삼기 위함이었다. CheckStyle 실행 결과 발생한 1,823개의 경고 중 상당 부분을 제거했으며, 나머지 경고는 점진적으로 처리할 계획이다.
주요 수정 작업
구체적으로 수행된 작업은 다음과 같다:
- 중괄호 생략 금지:
if문 등에서 중괄호를 생략한 5건의 위반 사례를 수정했다. - 공백 추가:
if,for,while,try키워드 뒤 괄호에 공백을 추가하고, 화살표 연산자(->) 좌우에 스페이스를 넣었다. - 변수명 수정: 대문자로 시작하는 lambda 변수 이름을 소문자로 변경했다.
- 탭 문자 교체:
.java파일 내 탭 문자를 2개의 스페이스로 일괄 교체했다.
이러한 작업은 모두 sed와 ag 도구를 활용해 효율적으로 수행되었으며, 각 변경 사항은 Pull Request를 통해 머지되었다. 결과적으로 프로젝트의 코드 컨벤션 준수율이 높아졌으며, 팀원들의 신중한 Git 커밋 메시지 작성과 코드 리뷰 프로세스가 자리 잡는 데 기여했다.
이 한국어 요약은 AI가 자동으로 만들었습니다. 원문의 주장과 맥락은 원문에서 확인해 주세요. 저작권은 원저작자에게 있습니다.