-
Notifications
You must be signed in to change notification settings - Fork 1
Team Rule
kory0115 edited this page Jul 1, 2024
·
10 revisions
팀 규칙
-
코드 리뷰는
PR이 올라온 후 24시간 이내에 request Change 또는 이유 있는 승인 코멘트를 남기기코드 리뷰를 남기고, 응답하는 데 소요하는 시간은최대 1시간 이내로 마무리한다. -
PR을 올리면 PR 코드 리뷰 전용 카톡방에 올리기
-
24시간 이내에 코드 리뷰를 남기기 어려우면 카톡방에 남기기
-
PR 올리기 전에
에러가 없는지 반드시 실행을 통해 검증해야 하며, dev branch에 있는 코드를 받아와서 최신화 한다. -
코드 리뷰 코멘트 남길 때,
필수/선택으로 나눠서 작성하기 - 가능하면 ref 남기기-
필수: 수정하지 않으면Error가 발생할 수 있는 코드, UI/UX 경험을 해칠 수 있는 코드, 협의된 내용과 다른 로직이나 디자인) -
필수-선택 중간쯤: 프로젝트구조에 적합하지 않은 부분 -
선택: 좀 더 높은효율성(참고할 수 있는 사이트를 같이 제공)을 제공할 수 있는 부분
-