ci(actions): skip draft pull request jobs - #147
Conversation
Signed-off-by: Seongho Bae <me@seonghobae.me>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughCI 워크플로가 지정된 pull request 이벤트에서만 실행됩니다. 초안 또는 종료된 pull request에서는 세 작업을 건너뜁니다. 계약 테스트가 이벤트 목록과 작업 조건을 검증합니다. ChangesCI pull request 실행 조건
Estimated code review effort: 2 (Simple) | ~10 minutes ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4c056aaee3
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| push: | ||
| branches: [main] | ||
| pull_request: | ||
| types: [opened, synchronize, reopened, ready_for_review, converted_to_draft, closed] |
There was a problem hiding this comment.
Document the new draft-PR CI behavior
This changes externally observable automation behavior by suppressing all three CI jobs for draft and closed pull requests, but the commit leaves CHANGELOG.md, the CI/operator documentation, and the doctoring record unchanged. Document when these checks are skipped and rerun—especially that skipped jobs are not exact-head verification—so maintainers and automation do not misinterpret draft-head evidence; repository policy requires these documentation updates whenever behavior changes.
AGENTS.md reference: AGENTS.md:L56-L57
Useful? React with 👍 / 👎.
요약
검증
actionlint .github/workflows/ci.ymluv run --locked --project services/account_unification --extra dev pytest -q services/account_unification/tests/test_workflow_concurrency_contract.py→3 passed, 경고 없음전체 actionlint의 기존 hourly-product-development SC2016은 #146이 해당 직접 NIM loop를 삭제하며 소유합니다. 이 PR은 Actions 60-job ceiling 수리만 포함합니다.
Summary by CodeRabbit
CI 개선
테스트