Skip to content

나아가의 PR과 코드리뷰 규칙

k_dragonm edited this page Sep 5, 2023 · 1 revision

리뷰

  • PR에 정해진 D-day 태그를 지켜서 리뷰해야해요!
  • 리뷰할 때 자신의 말하기 강도를 Pn 전략으로 표현해요

Pn 전략

n은 1에서 5까지 존재합니다.

  • P1: 꼭 반영해주세요 Request Change
  • P2: 적극적으로 고려해주세요 Request Change
  • P3: 웬만하면 반영해 주세요 Comment
  • P4: 반영해도 좋고 넘어가도 좋습니다 Approve
  • P5: 그냥 사소한 의견입니다 Approve

PR

  • 한번 PR을 올리는 단위는 200~300 줄의 코드라인 변경 (초록색 변경만 해당)
    • PR을 작게 유지해야 리뷰하기 좋아요!
  • PR을 올릴 때 D-day 태그를 붙여요!

D-day 태그

  • PR을 올리는 사람이 생각하는 마감 기한을 뜻해요.
  • day는 0에서 2까지 존재합니다.
  • 프로젝트에 대한 영업시간은 18시까지이며 18시 이후의 PR은 자신이 생각하는 것에 +1을 해요.
  • 매일 아침 데일리 미팅 시간에 본인 PR의 D-day 태그를 갱신해요.