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

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

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

두줄요약

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

문제 상황

  • 팀 코딩 스타일 가이드 위반 여부 확인 중 중괄호 생략 사례와 CheckStyle 경고 1,823개 발견
  • 신규 입사자의 프로젝트 기여와 코드베이스 이해를 위한 개선 과제

해결 방법

  • find, egrep, ag로 규칙 위반 코드를 검색하고 sed로 제한적·기계적 수정
  • 공백, lambda 변수명, 화살표 연산자, 탭 문자, 중괄호 앞 공백 등을 작업 단위별 PR로 분리
  • 테스트 확인과 변경 파일 검토 후 PR 제출·리뷰·머지 진행

선택 이유

  • 일괄 포매터 적용 대신 작은 PR을 통한 GitHub·커밋·브랜치·코드 리뷰 학습 기회 활용
  • 코드 컨벤션 준수의 중요성 공유와 점진적 코드베이스 정리

성능/운영 포인트

  • 보이스카웃 규칙에 따른 발견 즉시 경고 제거 방식
  • 500개 이상 경고 제거와 신규 프로젝트의 낮은 경고 수준 유지
  • PR·코드 리뷰·신중한 커밋 메시지 중심 개발 프로세스 정착

다음 읽기

#Java 주제를 이어서 읽기

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

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

마켓컬리
마켓컬리
백엔드

댓글 0개

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

댓글을 불러오는 중...