목록 보기
새로 입사한 개발자가 프로젝트에 기여하는 방법 한 가지
백엔드

새로 입사한 개발자가 프로젝트에 기여하는 방법 한 가지

마켓컬리
마켓컬리
2020년 3월 9일

두줄요약

신규 입사자가 CheckStyle 경고를 작은 PR 단위로 줄이며 프로젝트에 기여한 사례를 소개했습니다.\nsed 기반 일괄 수정과 코드 리뷰를 학습 기회로 활용해 컨벤션 문화를 강화했습니다.

문제 상황

  • 신규 입사 후 코드 컨벤션 위반 여부를 확인하며 발견한 프로젝트 전반의 CheckStyle 경고 1,823건
  • 중괄호 생략 등 팀 규약 위반 코드와 누적된 포맷 불일치

해결 방법

  • Java 파일 검색 결과를 바탕으로 작은 단위의 컨벤션 수정과 Pull Request 반복 제출
  • ag, find, sed를 활용한 괄호·람다 변수명·화살표 공백·탭·중괄호 공백의 기계적 수정
  • 변경 후 git diff와 테스트 확인, 백업 파일 제외 후 커밋

선택 이유

  • 전체 포매터 일괄 적용 대신 PR·코드 리뷰·커밋 단위 학습 자료로 활용
  • 코드 컨벤션 준수의 의미를 공유하고 코드베이스 이해를 병행하는 점진적 접근

성능/운영 포인트

  • 500개 이상 경고 제거와 신규 프로젝트의 1개 이하 경고 유지
  • PR 리뷰와 머지 중심의 협업 프로세스, 신중한 커밋 메시지 문화 정착

다음 읽기

#Java 주제를 이어서 읽기

새로 입사한 개발자가 프로젝트에 기여하는 방법 한 가지

코드 컨벤션 경고를 작은 단위의 PR로 점진적으로 제거한 경험을 소개했습니다.\n자동 수정과 리뷰 과정을 학습 기회로 활용해 팀 개발 문화를 강화했습니다.

마켓컬리
마켓컬리
백엔드

댓글 0개

댓글을 작성하려면 로그인이 필요합니다.

댓글을 불러오는 중...