Skip to content

그라운드 룰

Park-Mu-Seong edited this page Jan 10, 2025 · 6 revisions

🌱커뮤니티 생활 규칙

  • 코어 수면시간 지키기
    • 밤새면 커피 사기
  • 참여 불가 날짜 미리 달력에 적어두기
  • 참여 불가 날 하루 전에 미리 알려주기
  • 경어 사용하기
  • 말투 둥글둥글

🏠디스코드 생활 규칙

  • 코어타임 동안 디스코드 개인 룸 접속해있기(13시 ~ 18시)
  • 자리 비움 시 마이크 OFF, 채팅에 자리 비움 시간 알리기
  • TTD(Today’s To Do) 채널
    • 당일 할 일 목록을 해당 일 오전 09:00시 이전에 업로드 (전날 업로드 하고 자야 함.)
대화 가능 대화 가능
사용자가 응답 가능 상태입니다.
휴식 중 휴식 중 또는 자리 비움
사용자가 현재 자리를 비웠거나 휴식 중입니다.

🌍기술공유 규칙

  • 매주 금요일 스프린트 리뷰
  • 부스트캠프 리팩토링 기간 동안 한 사람 당 기술 공유 하나 이상 참여 필수 [기술 공유 문서 모음]
  • 노션이아니라 개인 블로그에 작성
  • 스프린트 리뷰 시간을 더 의미있게 만들고 경험을 함께 공유하기 위해 시작했습니다.
    • 처음에는 글쓰는 게 어려울 수도 있으니 비어있는 글이나 미완성도 자유롭게 올리기로 했습니다.
    • 프로젝트와 관련된 주제가 아니더라도 본인이 흥미있는 주제 등을 선정하여 작성했습니다.
    • 모든 팀원이 들으면서 언제든지 의견과 생각을 공유하는 시간으로 활용했습니다.

image

⚖️의사 결정 규칙

  • 의견 수립은 기본적으로 다수결을 따릅니다.
    • 하지만 반박이 나올 시 의견을 꼭 들어야 합니다.

💪컨디션 조절 규칙

  • 50분 집중, 10분 휴식
    • 주말에 하루는 쉬는 날
    • 팀원 건강 체크인 (매일 데일리 스크럼에 본인 컨디션을 언급하고 시작합니다.)

✅코드리뷰 규칙

  • PR 템플릿에 맞춰서 꼼꼼하게 작성합니다.
  • 너무 사소한 경로 변경, 글자 하나 고치기 정도는 그냥 Commit Message에 내용 포함하고, PR 내용 다 지우고 올려도 됩니다.

✍️ PR 룰 도입

  • 네이버 부스트 캠프 정규 기간 PR 리뷰를 할 때 체력 소모가 꽤 있었던 것을 확인했습니다.
  • 작성한 PR의 목적성을 뚜렷하게 나타내고 PR과 관련 없는 작업을 막고자 PR 룰을 도입했습니다.
    • 추가적으로 코드 리뷰어가 비동기적으로도 쉽게 협업할 수 있는 환경을 제공합니다.
  • Line Changes 400줄 이하(추가, 삭제 포함)으로 PR을 작성하도록 합니다.
  • 작업상 특이 케이스나 미미한 차이 정도는 유동적으로 봐주기로 합니다.

🗣️Pn 룰 도입

  • 뱅크샐러드의 Pn룰을 모방하여 PR을 효과적으로 리뷰하기 위해 Pn룰을 도입했습니다.
  • 모든 리뷰어들은 우선 순위에 따라 P5 ~ P1을 명시하고 피드백을 작성합니다.
우선순위 설명 예제
P5
(질문 및 추천)
제안사항으로, 반드시 반영할 필요는 없습니다. P5) 이런 식으로 리팩토링하면 더 간결해질 수 있을 것 같아요.
P4
(가벼운 제안)
가급적 고려되길 권장하지만 반드시 수정할 필요는 없습니다. P4) 변수명을 조금 더 명확하게 바꿔보는 건 어떨까요?
P3
(중요)
비교적 기능에 영향을 미칠 수 있는 가능성이 미미하게 존재해 수정을 적극적으로 고려해야 합니다.
중요한 개선 사항이거나 모호한 부분에 대한 질문입니다.
P3) 이 로직은 경계값 테스트를 추가하는 게 좋아 보여요. 의견 주세요.
P2
(매우 중요)
코드 품질이나 기능에 영향을 미칠 수 있는 사항으로, 반드시 반영해야 합니다.
P2부터는 반드시 리뷰어와 반영 여부를 논의후 결정하여야 합니다.
P2) 여기서 무한 루프 가능성이 있습니다. 수정이 필요합니다.
P1
(최우선)
즉각 수정해야 할 중대한 문제로, 배포 시점 전에 반드시 해결되어야 합니다. P1) 이 부분은 비즈니스 로직이 잘못 구현되었습니다. 수정하지 않으면 심각한 버그가 발생할 수 있습니다.

저희만의 시행착오를 겪고 P3 이하의 코멘트들은 작성자가 Resolve하는 규칙을 추가했어요.

image image

저희는 이렇게 활용하고 있어요

프론트엔드 대표 PR 보러 가기
백엔드 대표 PR 보러 가기

image

그라운드 룰 업데이트 로그