코드 리뷰

👍 PR 올리기 전에?

• 실행 가능 한 상태로 올려주세요

🏃‍♂️ 코드 리뷰는 24시간 내로

• 목표는 하던 작업 마무리 하고 바로 할 것

• 리뷰는 기다리고 있는 팀원을 위해 최대한 빨리 해주기

• 말을 하거나 Slack 알림 설정, Jira 티켓을 코드리뷰로 옮겨 알림을 할 것

• 리뷰는 다 하면 좋지만 최소 2명의 approve 받으면 넘어가도록 유연하게 운영하기

• 리뷰 반영 후 리뷰어가 resolve conversation 을 모두 눌러주면 머지
• resolve 또한 PR룰과 마찬가지로 운영

• resolve 플로우
• 리뷰 한 번만 하고 리뷰 반영은 PR 작성자가 선택
• 리뷰 반영 후 반영 사항 리뷰 요청
• 리뷰어는 반드시 반영 사항 확인 후 approve
• PR 작성자는 머지 방식에 따라 머지 진행

🔖 PR 내용은?

• PR 템플릿에 맞춰서 올려주세요

• 사소한 것, 궁금한 것, 해결해야하는 요소에 대해서 얘기해주세요!

• 커밋 분리를 최대한 시도해봐요

🏕️ 머지 방식

git switch develop
# develop branch 최신화
git pull origin develop
# 작업 branch 예시 : DK-104
git switch DK-104
# rebase
git rebase develop
# 만약 conflict 발생 시 해결 후
git add .
git rebase --continue
# 위 과정 반복 그리고 rebase 종료 후
git push origin DK-104 -f
# 그리고 github 가서 merge 버튼 클릭!

🎲 작업 브랜치 최신화 방법

# 작업 코드가 남아있다면
git add .
git stash
# 이후 머지 코드와 동일
git stash pop
# 만약 conflict 발생 시 해결 후
git add .
# stash에 남아있기 때문에
git stash drop
# staged 상태를 풀어준다
git restore --staged .
# 그리고 작업을 이어가면 된다!

🌟 유용한 깃 명령어

• git fetch --all --prune
• 가끔씩 리모트 브랜치를 삭제했는데 로컬에 남아있는 경우가 있어 그래프 확인 시 헷갈릴 수 있다
• 이 명령어는 리모트와 로컬의 상태를 동기화 시켜준다

📃 PR 템플릿

• 과제 때 쓰던 PR 템플릿 사용

## 📌 개요 <!-- 어떤 걸 만들었는지 대략적으로 설명해주세요 --> 

## 👩‍💻 구현 내용 <!-- 기능을 Commit 별로 잘개 쪼개고, Commit 별로 설명해주세요 --> 

## ✅ 피드백 반영사항  <!-- 지난 코드리뷰에서 고친 사항을 적어주세요. 재PR 시에만 사용해 주세요! (재PR 아닌 경우 삭제) --> 

## ✅ PR 포인트 & 궁금한 점 <!-- 리뷰어 분들이 집중적으로 보셨으면 하는 내용을 적어주세요 -->