swppfall2019
swppfall2019 copied to clipboard
[small tip]Protecting your master branch
안녕하세요, team7 유진선입니다.
어제 Travis CI 실습하면서 "앗 이걸 다른 브랜치에서 한번 시험해보고 master에 옮기면 앞으로 모든 branch에서 쓸 수 있겠지!"라고 생각했는데요
제 실수로 저렇게 바로 pull 하지 말라고 써놓고서는 그걸 master
로 그대로 밀어버렸습니다...
앞으로 이런 일이 일어나지 않도록 알아본결과, "Master로 바로 push하지 못하도록 막는 방법"을 찾아, 혹시나 모르는 팀이 있으면 유용하게 쓰실 수 있을 것 같아, 이렇게 공유하고자 합니다.
- 리포지토리로 가서 세팅 -> 브랜치 -> add rule로 갑니다
2. 위에 표시된 옵션을 세팅한다
-
master
브랜치로 바로 푸시하려고 시도하고, 안된다는 것을 깨닫는다. - 나의 실수로 팀의 master 브랜치를 꼬아버릴 걱정을 던다.
이렇게 하면 master
브랜치로는 바로 push를 할 수 없고, 다른 브랜치를 만든 다음에 반드시 1명 이상의 review를 받아야 master
브랜치에 변동이 생기게 됩니다.
master대신에 다른 branch도 이렇게 설정할 수 있으며, dev/*
와 같이 특정 기능의 여러 브랜치에 규칙을 적용할 수 있습니다.
저렇게 하면 master로 PR을 할때마다 누군가의 review를 받아야 되는데, 급한 경우에는 어떻게 하냐! 는 질문에 대해서는
- 모범답안: master 로 PR하는 것이기 때문에 review를 거치는 것을 권장드립니다
- At Your Own Risk답안: 저희 모두 repo의 administrator이기 때문에 몇 가지 warning만 무시하면 바로 merge시켜버릴 수 있습니다.
모두 즐거운 HW3, Sprint 3되시길 바랍니다 Team 7 일동.