[🎀깃 & 깃허브] pull-request 협업 방법

하나의 원격 저장소(메인 repo)에서 협업하기 위해서, 각자의 지역 저장소(repo)에서 작업 후, PR하여 원격 저장소에 공유하고 합치는 작업

---

원격 저장소(메인 repo)에 있는 코드들 최신 상태로 유지하며, 자신의 지역 repo로 가져와서 로컬에서 수정 후, PR (pull-request) 보내는 과정 반복

🎁 리액트 git 협업 환경 세팅

1. 만들어진 원격 저장소 레포 내 레포로 fork 해오기

-> 원격 저장소에서 fork 버튼을 눌러준다.

2. cmd 창 열기

3. cd Desktop: 바탕화면에 프로젝트 파일 만들기위해 바탕화면으로 이동

4. git clone "원격 저장소 깃허브 주소" "프로젝트 이름": 원격 저장소 내용 clone 해오기 (바탕화면에 프로젝트 생성)

-> "프로젝트 이름"으로 바탕화면에 프로젝트 만들고, "원격 저장소 깃허브 주소"의 내용 clone 해옴

5. cd "프로젝트 디렉토리": 그 프로젝트 디렉토리로 이동

6. code .: vs code 파일 열기

7. git remote add "이름" "원격 저장소 주소": 원격 저장소 remote하기

8. remote -v: remote 잘 되었는 지 확인

9. git fetch upstream(7번의 이름) main(브랜치 명), git rebase upstream/main 또는 git pull

-> rebase시 위와 같은 오류가 뜬다면, 내가 작업하고 있는 상태에서 원격 저장소의 내용 가져와서 생겨나는 오류이다. 따라서, 일단, 내가 작업하고 있는 것을 스택에 넣어놓고, 원격 저장소의 내용을 먼저 가져온 다음에, 내가 하던 작업을 다시 꺼내오면 된다.
-> fetch와 rebase를 하거나 pull을 해도 된다. 나는 fetch와 rebase를 이용하는 방법으로 협업하였다.
✔️ 해결 방법:

• git stash: 내가 하던 작업 사라짐
  -> git rebase upstream/main
  -> git stash pop
  -> 충돌 해결

10. git checkout -b "브랜치 명": 브랜치 생성 후, 그 브랜치로 이동

11. 내 코드 작성 후, 나의 레포 10에서 생성한 브랜치에 업로드

-> 팀장이라면, npx create-react-app 해서 push 해놓기

• git add .
  -> git commit -m "커밋 메시지"
  -> git push origin "10에서 생성한 브랜치 명"
  

12. 나의 레포에서 pull-request 하기

13. 팀원들한테 pr 보냈다고 알리기

13.1. 팀원들이 pr 했다고 하면, 원격 저장소의 pull-request의 'file chages'에 코드 리뷰해준 후, 알리기

14. 팀원들이 코드리뷰 해주면, 원격 저장소에서 확인 후, 원격 저장소 main에 merge하기

15. 9~14 계속 반복하기!!!!

---

😀 repo 생성 방법

1. repository 생성

2. 공동 작업자 추가하기

: 해당 repo로 들어가서, Settings -> Manage access -> Invite a collaborator

: username이나 email등을 이용하여 초대 9초대받은 팀원들은 메일이나 깃허브 메시지 화면에서 확인하여 accept 누르면 됨)

3. 위의 협업 세팅 과정 똑같이 따라하고, CRA해서 push해서 기본 리액트 세팅해놓기

---

😀 팀원 간 코드 리뷰하기

• 각 팀원이 PR 한 것은 원격 저장소(메인 repo)에서 확인 가능
• 공동 작업자 누구나 merge 가능
• 깃허브 협업 시, 변경 내용을 바로 반영하기 위해 항상 pull을 먼저 당긴후에 자신의 작업을 진행하는 것을 권장
• merge는 모든 팀원들이 할 수 있지만, 보통 본인이 책임진다는 마인드로 PR을 올린 본인이 Merge

1. PR 살펴본 후, 이상이 없으면 merge하기 (confirm merge)
2. 브랜치 병합되면, master 브랜치에 모두 나타남
   
   : files change에서 코드리뷰 남겨준다.
   
   -> 코드리뷰 완료 후 체크표시가 뜬다.
   
   -> merge 완료

🎁 코드 리뷰 태그

1. LGTM

: Looks Good To Me
-> 코드에 별 문제가 없다.

2. AFAIK

: as far as I know
-> 내가 알기에는, 내가 기억하는 한

3. IMO

: in my opinion
-> 내 생각에는

4. IMHO

: in my humble opinion
-> 제 견해에는 요 (IMO보다 조금 더 격식 있는 표현)

5. SGTM

: Sounds Good to Me
-> 좋다. 1번이랑 같은 뜻

6. PTAL

: Please Take Another Look
-> 제발 좀 봐주세요. 리뷰어가 리뷰를 해주다가 중간에 stop된 상태에서 리뷰좀 해달라고 부탁할때 쓰는 것

7. FYI

: For Your Information
-> 참고로

8. ASAP

: as soon as possible
-> 가능한 빨리, 최대한 빠르게

9. IOW

: In Other Words
-> 다른 표현으로, 다시 말하면, 바꾸어 말하면

10. TL;DR

: Too Long. Didn't Read
-> 보통 문장 앞에 긴 글을 요약할 때

---

😀 fork 없이 협업하기

: 브랜치 생성 => 작업 => PR => 코드 리뷰 => merge

1. repository 생성

2. git clone으로 모든 팀원들이 본인 로컬에 프로젝트 복제

vs code: 컴퓨터에 디렉토리 생성 후, 원격 저장소 주소를 복사하여, F1 을 누르고 git clone을 작성한 후 url 붙여넣기

3. 같은 레포에 새로운 브랜치 생성

: 보통 main 브랜치에서 새로운 브랜치를 따고, 여기서 작업한 후 Pull Request(이하 PR)를 날려 코드리뷰 후 merge
-> 메인이 계속 바뀌면 다른 작업자는 원격과 다른 환경에서 개발을 진행하게 되어 수 많은 충돌을 맞게 될 것 이며
엄청난 비효율

4. git pull

: 변경 사항 가져오기 (첫 commit 아니라면)

5. 3의 브랜치에 코드 수정 후 add, commit, push 하기

-> 그러면, 새로운 브랜치가 생성되고, 그곳에 수정된 내용이 반영된다.

6. pull-request

-> 해당 브랜치 이름을 누르면, pull-request 하겠냐는 메시지

: 위에 base에 원하는 브랜치가 넣어져 있는지 확인한 후, 오른쪽에 팀원을 리뷰어로 선정하고, 정성스럽게 PR을 작성하고, Create pull request

-> 그럼 이와 같은 화면
-> PR 날린 후, 다른 팀원들에게 코드 리뷰 요청하기

------- 또 다른 팀원은 ------

다른 팀원의 코드 리뷰

: Files changed로 가서 리뷰를 해주고

: 모든 리뷰를 끝냈다면 우측 상단에 Finish your review를 누르고 Approve를 누르고 Submit review를 누르면 리뷰 완료
---------------- 다시 나는 ------

7. 코드 리뷰 확인과 MERGE

-> Reviewers의 리뷰가 끝나면(Approve가 되면) 오른쪽 상단처럼 체크표시
-> (merge는 모든 팀원들이 할 수 있지만, 보통 본인이 책임진다는 마인드로 PR을 올린 본인이 Merge한다.)

-> merge 이후 다른 팀원들에게 pull 요청

8. MERGE 확인

: 메인 브랜치에서 변경 사항 확인 가능

✔️ 정리:
하나의 원격 저장소에 협업을 하기 위해서는 팀원들이 모두 메인 원격 저장소를 fork 해오고, pull로 변경 내용 당겨온 이후, 각자 로컬에 브랜치 새로 파서 코드 수정 후, 각자의 로컬에서 add, commit, push 이후, remote로 연결된 원격 저장소에 PR을 보낸다. 그럼 다른 팀원들은 이를 확인하여 code review를 남겨주고, 그에 따라 merge를 해준다.

---

참고

• https://chanhuiseok.github.io/posts/git-3/
• https://inpa.tistory.com/entry/GIT-%E2%9A%A1%EF%B8%8F-%EA%B9%83%ED%97%99-PRPull-Request-%EB%B3%B4%EB%82%B4%EB%8A%94-%EB%B0%A9%EB%B2%95-folk-issue
• https://parkparkpark.tistory.com/176