swsnu / swpp2021-team12

🤝MeetHub🙋‍♂️🙋‍♀️ the place where people get together.
5 stars 0 forks source link

feature 브랜치에서 건드리지 말아야 할 부분을 정리합시다. #7

Open KylusheL opened 3 years ago

KylusheL commented 3 years ago

공통으로 관리하는 부분의 경우 conflict 이슈를 고려해 dev 브랜치에서 수정하도록 하고, feature 브랜치에서는 손대지 않아야 할 부분을 확실히 정하고 가야할 것 같습니다.

그때그때 생각날 때마다 여기에 추가합시다.

당장 생각나는 것들 먼저 적어보겠습니다.

index.js App.js store 폴더 내의 파일들 (actionTypes.js, action폴더, reducer폴더)

index, App같은 경우는 변경해야 할 사항이 있으면 의논해서 결정 후 dev 브랜치에서 수정하고

store 폴더같은 경우는 한 파일을 여럿이 건드려야 하는 경우가 있을 수 있기 때문에 겹치는 부분같은 경우는 파일을 각자 만든 후 dev에서 정리하는 식으로 가면 어떨까 싶습니다.

ex) 이의진과 강휘현이 store/reducers/UserReducer.js 를 건드려야 할 때 각 feature 브랜치에서는 UserReducer-Lee.js, UserReducer-Kang.js 로 파일을 각각 만든 뒤 dev에서 하나로 합쳐버리는 방식을 생각해봤습니다.

공동작업이 처음이라 미숙한 부분이 많으니 많은 피드백 바랍니다.

rkdgnlgus2 commented 3 years ago

conflict 이슈는 여러명이 같은 부분을 동시에 건드리면 발생하기 때문에 브랜치별로 나눠서 작업할 때는 크게 어려움이 있을 것 같지 않습니다. 만약 겹치는 부분이 있다면 pull request에서 직접 수정할 수 있기 때문에 그 때마다 이슈를 만들어 의논한 뒤 해결하면 될 것 같습니다. 이슈를 방지하기 위해 dev로부터 주기적으로 merge하며 충돌을 제거해도 좋을 것 같습니다. 참고가 될만한 레퍼런스입니다. https://backlog.com/git-tutorial/kr/stepup/stepup2_7.html https://jeong-pro.tistory.com/106