프로젝트/richman

Git PR(Pull Request) 개념부터 실전까지 한 번에 정리

minaz-rong 2026. 4. 29. 16:45

🔄 우리 팀 PR (Pull Request) 규칙

팀 프로젝트에서 PR은
👉 코드를 합치기 전 반드시 거치는 “검토 과정”입니다.
아래 규칙을 반드시 지킵니다.

 

🔄 언제 사용하나요?

  • feature 브랜치에서 작업 완료 후
    👉 develop 브랜치에 반영할 때 사용

🔄 어디서 작성하나요?

👉 GitHub 웹사이트에서 작성

경로:

  • Repository 접속
  • Pull requests 클릭
  • New pull request 버튼 클릭

 

📌 1. 기본 원칙

  • PR 없이 merge 금지 ❌
  • 모든 코드는 PR을 통해서만 develop에 반영 ⭕
  • PR = “내 코드 리뷰 요청”

 

📌 2. PR 방향 (중요 ⭐)

feature/* → develop

예:

feature/mina-chatbot → develop
feature/boo-stock-analysis → develop

 

📌 3. PR 작성 규칙

 

템플릿 (필수 사용)

## 작업 내용
## 변경 사항
## 리뷰 포인트
## 관련 이슈

 

작성 기준

  • 작업 내용을 한눈에 이해 가능하게 작성
  • 변경 사항은 핵심만 정리
  • 리뷰 포인트는 꼭 확인받고 싶은 부분 작성

진행 흐름

  1. feature 브랜치에서 작업
  2. commit & push
  3. PR 생성 (→ develop)
  4. 코드 리뷰
  5. 승인 후 merge

 

📌 4. 리뷰 & Merge 규칙

  • 최소 1명 이상 승인 후 Merge
  • 승인 없이 merge 금지
  • 리뷰 요청 시 팀원 태그 필수

👉 Merge 방식: Squash Merge 사용(권장)

 

📌 5. PR 크기 규칙 (중요)

  • 기능 단위로 PR 생성
  • 너무 큰 PR 금지
  • 하루 작업 단위로 PR 권장

❌ 나쁜 예

  • 기능 여러 개를 한 번에 PR

✅ 좋은 예

  • 로그인 기능 PR
  • 회원가입 기능 PR

 

📌 6. 작업 흐름

  1. feature 브랜치 생성
  2. 기능 개발
  3. commit & push
  4. PR 생성 (→ develop)
  5. 코드 리뷰
  6. 승인 후 merge

 

📌 7. 금지 사항

  • main 브랜치로 PR 금지
  • develop에서 직접 작업 후 merge 금지
  • 리뷰 없이 merge 금지
  • 의미 없는 PR 생성 금지

 

📌 8. 권장 사항

  • 하루 1회 이상 PR
  • 오래된 브랜치 방치 금지
  • 충돌 발생 시 직접 해결 후 PR

 

📌 9. 작성예시

 

예시 1. 챗봇 기능

## 작업 내용
크립토 챗봇 기본 기능 구현

## 변경 사항
- 뉴스 요약 기능 추가
- 감정 분석 결과 반환
- 챗봇 API 연결

## 리뷰 포인트
- 응답 속도 괜찮은지 확인 부탁드립니다
- 감정 분석 결과 정확도 체크해주세요

## 관련 이슈
- PROJ-12

 

예시 2. 주식 분석 기능

## 작업 내용
주식 데이터 수집 및 기본 분석 기능 구현

## 변경 사항
- 주가 API 연동
- 데이터 전처리 로직 추가
- 매수/매도 감정 계산 로직 구현

## 리뷰 포인트
- 감정 분류 기준이 적절한지 확인 부탁드립니다

## 관련 이슈
- PROJ-18

 

예시 3. 소비 분석 기능

## 작업 내용
소비 데이터 시각화 기능 구현

## 변경 사항
- 카테고리별 소비 그래프 추가
- 월별 소비 통계 계산 로직 구현

## 리뷰 포인트
- 그래프 UI/UX 자연스러운지 확인 부탁드립니다

## 관련 이슈
- PROJ-25

 

🔥 한 줄 정리

👉 “작게 만들고, 자주 올리고, 리뷰 받고 합친다”

👉  PR = “내 코드 합쳐도 되는지 팀에게 묻는 과정”