깃허브 프로젝트 세팅
General - Pull Requests
Squash Merge 만 가능하게 변경
오른쪽 버튼 ( Switch to another branch ) 통해 선택해서 변경 가능
Security - Code security and analysis
secret, keys, token 이 있을시 Github가 자동 경고
프로젝트 시작 가이드에 있으므로 당연히 enable
커밋이 직접 브랜치로 푸시 방지 -> PR 통해서만 병합 가능
Main 은 우선 열어놓은 상태 ( develop 에서 검증 후 불필요한 프로세스라 판단 )
최소 Approve 해야 하는 리뷰어 수
프론트가 최대 2명이므로 2명 지정 백엔드는 도의적으로 한 명 더 merge 할 때 까지 기다려야 함 ( 또는 논의해봐야 함 - 2명 정도면 괜찮지 않을까? )
- Dismiss stale pull request approvals when new commits are pushed : 새로운 커밋 PR추가시 이전에 받은 승인 무효화
- Require approval of the most recent reviewable push : 최신 커밋에 대해 리뷰어가 승인
빠른 사이클을 위해 피드백 한 내용은 승인 안해도 되게 확인
- Restrict who can dismiss pull request reviews : PR 리뷰 기각 사람 제한
- Require review from Code Owners : 코드 소유자로 지정된 사람이 PR 검토 & 승인
- Allow specified actors to bypass required pull requests : 특정 사용자는 요건 우회하게 허용
이들은 불필요하다고 생각
- 기본 브랜치 변경사항이 있을 때마다 PR작성자에게 브랜치 업데이트 하라는 제안
develop branch 에 다른 개발자가 push를 할때 다른 개발자의 PR이 develop branch 에 상태 반영하지 않는 경우 발생할 수 있다!
프로젝트에서 단일 프로젝트이므로 발생하는
프론트/백엔드 간 PR-리뷰 사이클 중 develop 브랜치 동기화 문제 방지 가능 기대
( 깃허브 프로젝트에서 알림 형식으로 뜸 )
모든 필수 리뷰 & 상태 확인 완료시 PR 자동 병합
코드 리뷰와 Approve 가 완료된다면? -> 누군가 직접 merge 할 필요 없이 바로 가능 할 것으로 기대
하지만, 프론트/백엔드 간 최소 리뷰어 수가 다르므로 사용 X ( 백엔드는 3명이 리뷰해줘야 하나, 최소치가 2명이면 자동 Approve )
PR이 병합된 후 헤드 브랜치 자동 삭제
어차피 삭제할것이므로 자동 정리 효과 기대