🗯️ 문제
저번주에 PR을 Merge하는 과정에서 꼬인 부분이 있어 PR을 Close하고 다시 커밋하고 올리는 과정에서 develop 브랜치에서 작업하고 push하는 실수를 범하고 말았다.
프로젝트 초기에 브랜치 보호 규칙을 설정할 때, Require a pull request before merging (머지 전, pull request 를 해야한다.) 규칙을 적용했음에도 불구하고 바로 push가 되었다.
- Require a pull request before merging 체크되어 있는 부분
🔥 해결 방법
다시 한번 구글 검색을 통해 알아보니 브랜치 보호 규칙 중 Include administrators 라는 규칙이 있었다. 이 규칙은 레포지토리의 관리자도 브랜치 보호 규칙을 적용시키는 것을 의미했다. 아마 관리자는 모든 권한을 갖고 있기에 브랜치 보호 규칙에서 제외되는게 기본 설정인 것 같았다.
위 규칙을 찾아보고 우리 레포의 접근 권한을 찾아보니 세 명의 팀원이 모두 Admin으로 설정되어 있었다.
- Repository 접근 권한
그렇다면 내가 범했던 실수가 재발하지 않기 위해선 어떤 해결 방법이 있을까?
- Role을 변경
- Include administrators 규칙 적용
아무래도 Role을 변경하기엔 레포의 설정을 하는 부분에서 불편하고 제약사항이 많을 것이라는 생각이 많이 들었다. 그래서 Role은 유지한 채 Include administrators 규칙을 적용하는 것으로 결정하였다.
- Include administrators 적용
