🔄 우리 팀 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 작성 규칙
템플릿 (필수 사용)
## 작업 내용
## 변경 사항
## 리뷰 포인트
## 관련 이슈
작성 기준
- 작업 내용을 한눈에 이해 가능하게 작성
- 변경 사항은 핵심만 정리
- 리뷰 포인트는 꼭 확인받고 싶은 부분 작성
진행 흐름
- feature 브랜치에서 작업
- commit & push
- PR 생성 (→ develop)
- 코드 리뷰
- 승인 후 merge
📌 4. 리뷰 & Merge 규칙
- 최소 1명 이상 승인 후 Merge
- 승인 없이 merge 금지
- 리뷰 요청 시 팀원 태그 필수
👉 Merge 방식: Squash Merge 사용(권장)
📌 5. PR 크기 규칙 (중요)
- 기능 단위로 PR 생성
- 너무 큰 PR 금지
- 하루 작업 단위로 PR 권장
❌ 나쁜 예
- 기능 여러 개를 한 번에 PR
✅ 좋은 예
- 로그인 기능 PR
- 회원가입 기능 PR
📌 6. 작업 흐름
- feature 브랜치 생성
- 기능 개발
- commit & push
- PR 생성 (→ develop)
- 코드 리뷰
- 승인 후 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 = “내 코드 합쳐도 되는지 팀에게 묻는 과정”
'프로젝트 > richman' 카테고리의 다른 글
| 🚀 Jira + GitHub 협업 규칙 (0) | 2026.05.14 |
|---|---|
| 협업을 위한 폴더 구조와 네이밍 규칙 총정리 (0) | 2026.04.29 |
| 팀 프로젝트 시작 전, 반드시 정해야 할 협업 규칙2 (폴더구조, 네이밍 컨벤션) (0) | 2026.04.29 |
| 팀 프로젝트 시작 전, 반드시 정해야 할 협업 규칙1 (Git) (0) | 2026.04.29 |