반응형

Branch 2

[Git] PR merge 조건을 Git Branch Protection Rule으로 적용해보자

PR을 Merge할 때, 특정 조건을 만족해야 머지가 가능하도록 하는 방법은 없을까? github에서 간단한 설정만 해주면 가능하다! 위 레포에는 최소한 2명 이상의 code reviewer의 approved가 있어야 머지해줄 수 있도록 해주었다. 뿐만 아니라, merge 종류는 squash and merge만 허용해주도록 변경해주었다. 어떻게 설정해준 것일까? Git Settings General 레포지토리의 Settings - General에 들어가서 아래로 쭉 내리면 여러 기능들을 설정할 수 있는 것이 보인다. Feature에서는 Issues, Projects, Discussions를 만들지 말지 결정할 수 있다. 아마 초기 세팅하면 Discussions는 생기지 않을텐데, 여기서 체크표시해주면 r..

Infra/Git 2022.05.06

[Git] PR을 보낼 때엔 main 외의 별도 브랜치 생성 후 보내주자 (feat. git rebase merge 충돌 방지)

알고리즘 초급 스터디, 기업코테 스터디에서 github을 이용하여 코드를 커밋하고 코드리뷰하고 있다. 이 때 나는 스터디원들이 PR을 보낼 때, main 브랜치가 아닌 다른 브랜치를 생성시킨 후, 그 브랜치에서 PR을 보낼 것을 권장하고 있다. 브랜치를 확인해보면 main에서 바로 올리신 것 같아요. 이 경우에, squash and merge가 이루어진 후, 다음 주차 과제를 진행하기 위한 fetch 또는 rebase 과정 중에 충돌이 발생할 수 있을 것 같아요. week9와 같이 브랜치를 새로 생성해서 만드는 건 어떤가요? 물론 로컬 폴더 삭제 후, git clone을 하는 방법이라든지, 별도의 default branch를 생성해두었다면 위 충돌은 겪지 않을 것 같아서 더 좋다고 생각하는 방법으로 진행..

Infra/Git 2022.05.06
반응형