Skip to content

ci(platform): Viewed 자동 머지가 리뷰 상태를 보지 않게 한다 (#2277) - #2280

Merged
OnlyReadOneBook merged 1 commit into
developfrom
ci/2277-review-agnostic-auto-merge
Oct 5, 2026
Merged

OnlyReadOneBook merged 1 commit into
developfrom
ci/2277-review-agnostic-auto-merge

Conversation

@OnlyReadOneBook

Copy link
Copy Markdown
Collaborator

Closes #2277

무엇을 바꾸나

#2278 은 승인 0건 룰셋에 맞춰 reviewDecision null 을 통과시키되 CHANGES_REQUESTED·REVIEW_REQUIRED 는 계속 막았다. 리뷰를 폐지했으므로 Viewed 자동 머지가 리뷰 상태를 아예 보지 않게 한다. 개인 머지 스크립트(merge-sweep.sh)도 같은 기준으로 이미 바꿨다.

  • find-viewed-auto-merge-targets.mjs: 예약 판정에서 리뷰 상태 검사를 지우고, GraphQL 조회 필드에서 reviewDecision 을 뺀다.
  • 테스트: 예약·해제 모두 리뷰 상태(null·APPROVED·CHANGES_REQUESTED·REVIEW_REQUIRED)와 무관하게 Viewed 로만 가른다는 것을 고정한다.
  • viewed-auto-merge.yml 머리 주석과 README 머지 절에서 변경요청 언급을 뺀다.

검증

  • node --test .github/scripts/find-viewed-auto-merge-targets.test.mjs 17건 통과, .github/scripts 전체 통과
  • verify-markdown-links.mjs . 22개 파일 통과

CI Test Plan

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

Generated by Claude Code

리뷰를 폐지했으므로 변경 요청·승인 대기도 예약을 막지 않는다. 조회 필드에서 reviewDecision 을 뺀다.
Merged via the queue into develop with commit 333ce48 Oct 5, 2026
23 checks passed
@OnlyReadOneBook
OnlyReadOneBook deleted the ci/2277-review-agnostic-auto-merge branch October 5, 2026 13:39
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): 리뷰 승인 게이트와 리뷰 자동화를 걷어낸다

2 participants