Skip to content

[BOM-1182] feat: 2025-bom-bom PR 폴링 기반 리뷰어 배정 전환#41

Merged
Ryan-Dia merged 3 commits into
serverfrom
BOM-1182-리뷰어배정-폴링전환
Jul 5, 2026

Hidden character warning

The head ref may contain hidden characters: "BOM-1182-\ub9ac\ubdf0\uc5b4\ubc30\uc815-\ud3f4\ub9c1\uc804\ud658"
Merged

[BOM-1182] feat: 2025-bom-bom PR 폴링 기반 리뷰어 배정 전환#41
Ryan-Dia merged 3 commits into
serverfrom
BOM-1182-리뷰어배정-폴링전환

Conversation

@Ryan-Dia

@Ryan-Dia Ryan-Dia commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

📌 What

리뷰어 자동 배정의 감시 대상을 이 레포에서 woowacourse-teams/2025-bom-bom으로 전환합니다.

❓ Why

#40에서 도입한 배정 시스템이 이 레포(Bombom-Team/admin)의 PR에만 반응하도록 잘못 구현됐습니다. 실제 백엔드 리뷰가 일어나는 곳은 2025-bom-bom(base: server-dev, email-server-dev)이므로, 그 레포의 PR을 기준으로 배정해야 합니다. 단, 2025-bom-bom에는 파일을 추가하지 않고 모든 동작은 이 레포에서 실행합니다.

🔧 How

  • GitHub Actions는 다른 레포의 PR 이벤트를 직접 받을 수 없으므로 이벤트 기반 → 10분 간격 cron 폴링으로 전환
  • 워크플로우 3개(assign/complete/deadline)를 reviewer-rotation.yml 하나로 통합: 완료 동기화 → 신규 배정 → 기한 초과 알림 순서 보장
  • 배정 조건: 대상 브랜치의 열린 PR 중 OPEN 배정 기록 없음 + 수동 지정 리뷰어 없음 + 리뷰 없음 (수동 운영과 공존)
  • 한 실행에서 여러 PR 배정 시에도 로테이션이 이어지도록 로컬 상태 갱신
  • 2025-bom-bom 접근을 위해 BOMBOM_REPO_TOKEN(PAT) secret 신규 필요

👀 Review Point (Optional)

  • 머지 전 BOMBOM_REPO_TOKEN secret 등록 필요 (2025-bom-bom Pull requests Read/Write 권한 PAT)
  • cron은 default 브랜치(main) 기준으로만 동작하므로, server 머지 후 main 반영이 있어야 실제로 돌기 시작합니다
  • 이미 열려 있는 기존 PR들 중 리뷰어/리뷰가 없는 PR에는 첫 실행 때 일괄 배정됩니다 — 원치 않는 PR이 있다면 첫 실행 전에 수동 리뷰어를 지정해두면 스킵됩니다

@Ryan-Dia Ryan-Dia merged commit 875852a into server Jul 5, 2026
1 check passed
@Ryan-Dia Ryan-Dia deleted the BOM-1182-리뷰어배정-폴링전환 branch July 5, 2026 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant