Skip to content

ci(actions): stop Draft PR suite fan-out - #409

Merged
seonghobae merged 1 commit into
mainfrom
codex/ci-inactive-pr-admission
Sep 4, 2026
Merged

ci(actions): stop Draft PR suite fan-out#409
seonghobae merged 1 commit into
mainfrom
codex/ci-inactive-pr-admission

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

문제

Draft PR에서도 전체 JavaScript build/coverage, 세 브라우저 Playwright 설치·검증, Office Python job이 실행됐습니다. CI concurrency가 push를 ref 단위로 묶고 모든 이벤트를 취소 대상으로 삼아 non-PR evidence도 교체될 수 있었습니다.

변경

  • PR concurrency를 workflow·repository·PR 번호로 격리합니다.
  • push 실행은 github.run_id로 독립시킵니다.
  • converted_to_draftclosed가 기존 PR CI를 정리합니다.
  • Draft/closed에서는 세 고비용 job을 시작하지 않습니다.
  • ready_for_review에서 현재 HEAD 검증을 다시 시작합니다.

검증

actionlint .github/workflows/ci.yml
git diff --check
Test Files 156 passed (156)
Tests 882 passed (882)

Summary by CodeRabbit

  • CI 개선
    • 초안 상태이거나 종료된 풀 리퀘스트에서는 빌드, 테스트 및 릴리스 검증 작업이 실행되지 않습니다.
    • 풀 리퀘스트에서 최신 실행이 이전 실행을 자동으로 취소하도록 동시 실행 처리가 개선되었습니다.
    • 풀 리퀘스트 이벤트 감지를 위해 지원되는 활동 유형이 명확해졌습니다.

Co-Authored-By: Codex <noreply@openai.com>
Signed-off-by: Seongho Bae <me@seonghobae.me>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-04T21:39:22.014639Z 5fd7dc7 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@seonghobae
seonghobae merged commit 0b88c16 into main Sep 4, 2026
13 of 18 checks passed
@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 24a3b7ab-0956-4c38-baa6-b332597c8a46

📥 Commits

Reviewing files that changed from the base of the PR and between a40b948 and 5fd7dc7.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • src/workflowExactHead.test.ts

📝 Walkthrough

Walkthrough

PR 이벤트 유형과 동시 실행 조건을 조정했습니다. draft 또는 closed PR에서는 주요 CI 작업을 실행하지 않습니다. 관련 workflow 계약 테스트를 추가하고 갱신했습니다.

Changes

CI 실행 제어

Layer / File(s) Summary
PR 이벤트 및 작업 실행 조건
.github/workflows/ci.yml
PR 활동 유형을 명시했습니다. 비PR 실행의 동시 실행 그룹에 github.run_id를 사용합니다. 실행 취소는 PR 이벤트에만 적용합니다. build-and-test, browser-release-evidence, office는 draft 또는 closed PR에서 실행하지 않습니다.
CI 계약 테스트
src/workflowExactHead.test.ts
동시 실행 그룹과 cancel-in-progress 표현식을 검증합니다. PR 활동 유형과 세 작업의 draft 및 closed 제외 조건을 검증합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/ci-inactive-pr-admission

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae
seonghobae deleted the codex/ci-inactive-pr-admission branch September 4, 2026 21:37
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