swsnu / swppfall2019

31 stars 22 forks source link

[small tip]Protecting your master branch #169

Open jinsun-yoo opened 5 years ago

jinsun-yoo commented 5 years ago

안녕하세요, team7 유진선입니다.

어제 Travis CI 실습하면서 "앗 이걸 다른 브랜치에서 한번 시험해보고 master에 옮기면 앞으로 모든 branch에서 쓸 수 있겠지!"라고 생각했는데요 image

제 실수로 저렇게 바로 pull 하지 말라고 써놓고서는 그걸 master로 그대로 밀어버렸습니다...

앞으로 이런 일이 일어나지 않도록 알아본결과, "Master로 바로 push하지 못하도록 막는 방법"을 찾아, 혹시나 모르는 팀이 있으면 유용하게 쓰실 수 있을 것 같아, 이렇게 공유하고자 합니다. image

  1. 리포지토리로 가서 세팅 -> 브랜치 -> add rule로 갑니다

image

  1. 위에 표시된 옵션을 세팅한다

  2. master 브랜치로 바로 푸시하려고 시도하고, 안된다는 것을 깨닫는다.

  3. 나의 실수로 팀의 master 브랜치를 꼬아버릴 걱정을 던다.

이렇게 하면 master브랜치로는 바로 push를 할 수 없고, 다른 브랜치를 만든 다음에 반드시 1명 이상의 review를 받아야 master브랜치에 변동이 생기게 됩니다.

master대신에 다른 branch도 이렇게 설정할 수 있으며, dev/* 와 같이 특정 기능의 여러 브랜치에 규칙을 적용할 수 있습니다.

저렇게 하면 master로 PR을 할때마다 누군가의 review를 받아야 되는데, 급한 경우에는 어떻게 하냐! 는 질문에 대해서는

모두 즐거운 HW3, Sprint 3되시길 바랍니다 Team 7 일동.