-
Notifications
You must be signed in to change notification settings - Fork 4
Ground Rule
Taehyeon Kim edited this page Oct 12, 2022
·
3 revisions
PR
- merge x
- approve 인원 = 0
- 승찬
- 없어도 될 것 같다. 그냥 굳이? 어차피 할 것이다. 안 막아도 될 것 같다. 리뷰 반영하면 된다. 수정 시, approve 해야 함
- 반영 noti는 솔직히 카톡으로 하고 싶다.
- 이후에 대기해야하는 시간이 생겨서 비효율적이다.
- 단 리뷰는 필수.
- 하나 고치고 어푸릅 요청이 너무 별로다. 알아서 할 때이다.
- 은주씨
- 은주
- 코드리뷰 하면 좋을 것 같다.
- 굳이 블락까지는 필요하지 않다.
- 가짜서비스냐?
- 몰래 머지 없을 것 같다.
- protected rule o
- 냅다 푸시를 막자.
- 뱅크샐러드 코드리뷰 문화
- Pn룰
- Dn룰
- P1; 꼭 반영해주세요.(Request changes) / 무조건 반영O, 리뷰어 제시, 리뷰이 사실여부확인 + 답글
- P2; 적극적인 토론. / 리뷰이가 리뷰어의 의견에 코멘트를 남기고 적극적 토론.
- P3; 사소한 코멘트 / 의견을 안달아도 되고, 넘어가도 됨. 반영안해도됨.
p3;
의견 제시
- HOTFIX 같은 경우는 D-0(ASAP)으로 하면 좋겠다. ㄴ 모두가 동의하고 머지했는데 담당자가 생각해보니까 변경해야하는 부분이 생각남. ㄴ 이런 부분은 바로 반영 필요
- D-1;
- D-2;
- 스프린트 주기가 짧으니까 길 필요가 없다.