Skip to content

ci(platform): 리뷰 승인 게이트와 리뷰 자동화를 걷어낸다 (#2277) - #2278

Merged
OnlyReadOneBook merged 2 commits into
developfrom
ci/2277-null-review-decision
Oct 5, 2026
Merged

OnlyReadOneBook merged 2 commits into
developfrom
ci/2277-null-review-decision

Conversation

@OnlyReadOneBook

@OnlyReadOneBook OnlyReadOneBook commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

Closes #2277

무엇을 바꾸나

팀 리뷰를 머지 조건에서 뺀다. required-approval 룰셋(develop·main)은 승인 0건으로 내리고(저장소 파일이 아니라 설정이라 이 PR 밖에서 적용), 리뷰를 전제로 돌던 자동화를 저장소에서 지운다.

지운 것

  • 워크플로 4개: review-debt-guard(리뷰 빚이 있으면 새 PR 을 닫음), review-request-all(팀 전원 자동 리뷰 요청), latest-review-decision-event·latest-review-decision-reconcile(최신 리뷰 판정 리컨사일)
  • 스크립트·테스트: check-author-debt, label-awaiting-author-prs, reconcile-latest-review-decision, awaiting-author-policy·latest-review-decision-policy·review-debt-policy·review-debt-query 테스트
  • PR Label Reconcile(conflict-label.yml)의 awaiting-author 라벨 단계
  • Viewed Auto Merge 의 리컨사일 완료 트리거(주기 실행은 그대로)

고친 것

  • find-viewed-auto-merge-targets.mjs: 승인 0건이면 GitHub 가 reviewDecision 을 null 로 돌려주므로, 예약을 막는 상태를 CHANGES_REQUESTED·REVIEW_REQUIRED 로 좁힌다. Viewed 조건(전체 파일 Viewed, 덜 찬 PR 의 예약·큐 해제)은 그대로다.
  • ensure-api-quota 주석·테스트에서 지운 가드를 뺀다. 남은 게이트(merge-order-guard 등)의 quota 검사는 그대로다.
  • README 의 코드 리뷰 절을 머지 절로 바꾼다.

그대로인 것

  • 필수 체크 guard 는 merge-order-guard 의 잡이라 지운 워크플로와 무관하다.
  • PR 경유, 필수 검사, merge queue, 자동 어사인.

검증

  • node --test .github/scripts/*.test.mjs 729건 통과
  • verify-markdown-links.mjs . 22개 파일 통과
  • 지운 이름(review-debt-guard 등) 저장소 전수 grep 0건

CI Test Plan

{
  "androidTest": {
    "mode": "none",
    "reason": ".github 의 리뷰 자동화 워크플로·스크립트와 README 만 바꾼다. 앱 코드와 androidTest 경계 경로를 건드리지 않는다(ci-test-plan.mjs inspectAndroidTestImpact: full·selected 0건).",
    "tests": []
  }
}

required-approval 룰셋을 승인 0건으로 내리면 GitHub 는 reviewDecision 을 null 로 돌려준다. 예약 판정은 승인 대기(REVIEW_REQUIRED)와 변경 요청(CHANGES_REQUESTED)만 막는다.
#2277)

팀 리뷰를 머지 조건에서 빼면서 리뷰를 전제로 돌던 자동화를 함께 지운다. 워크플로 4개(review-debt-guard, review-request-all, latest-review-decision-event·reconcile)와 그 스크립트·정책 테스트, PR Label Reconcile 의 awaiting-author 단계, Viewed 자동 머지의 리컨사일 트리거를 뺀다. README 의 코드 리뷰 절을 머지 절로 바꾼다. 필수 체크 guard 는 merge-order-guard 의 잡이라 그대로다.
@OnlyReadOneBook OnlyReadOneBook changed the title ci(platform): 승인 0건 룰셋의 빈 reviewDecision 도 Viewed 자동 머지 대상으로 본다 (#2277) ci(platform): 리뷰 승인 게이트와 리뷰 자동화를 걷어낸다 (#2277) Oct 5, 2026
@OnlyReadOneBook
OnlyReadOneBook added this pull request to the merge queue Oct 5, 2026
Merged via the queue into develop with commit 534c5b1 Oct 5, 2026
28 of 30 checks passed
@OnlyReadOneBook
OnlyReadOneBook deleted the ci/2277-null-review-decision branch October 5, 2026 13:11
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.

maintenance(platform): 리뷰 승인 게이트와 리뷰 자동화를 걷어낸다

1 participant