Skip to content

ci(actions): consolidate pull request workflows - #144

Merged
seonghobae merged 1 commit into
mainfrom
codex/actions-queue-consolidation
Sep 4, 2026
Merged

ci(actions): consolidate pull request workflows#144
seonghobae merged 1 commit into
mainfrom
codex/actions-queue-consolidation

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

변경

  • CI concurrency를 workflow명, repository, PR 번호 단위로 묶고 PR 실행만 취소합니다.
  • non-PR 실행은 run_id로 격리해 서로 취소하지 않습니다.
  • 조직 필수 codeql-pr workflow와 중복되는 로컬 CodeQL workflow를 제거합니다.
  • 회귀 계약 테스트를 추가합니다.

검증

  • actionlint .github/workflows/ci.yml
  • uv run --project services/account_unification --extra dev pytest -q services/account_unification/tests/test_workflow_concurrency_contract.py — 2 passed
  • git diff --check

Identity/security 제품 코드와 release/publish/deploy/migration, hourly automation은 변경하지 않았습니다.

Summary by CodeRabbit

  • CI 개선

    • 풀 리퀘스트에서는 동일한 요청의 이전 실행만 자동 취소되며, 기타 이벤트 실행은 유지됩니다.
    • 이벤트 유형에 따라 실행을 구분해 불필요한 중복 실행을 줄였습니다.
  • 변경 사항

    • 저장소 내 CodeQL 워크플로가 제거되었습니다.
  • 테스트

    • CI 실행 취소 정책과 CodeQL 워크플로 구성에 대한 검증을 추가했습니다.

@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: cc012165-be87-4658-87c1-ee627379c14b

📥 Commits

Reviewing files that changed from the base of the PR and between 771224f and 7f43552.

📒 Files selected for processing (3)
  • .github/workflows/ci.yml
  • .github/workflows/codeql.yml
  • services/account_unification/tests/test_workflow_concurrency_contract.py

📝 Walkthrough

Walkthrough

CI 동시성 그룹과 실행 취소 조건을 변경했습니다. 로컬 CodeQL 워크플로를 삭제했습니다. 관련 동작을 검증하는 테스트를 추가했습니다.

Changes

워크플로 실행 제어

Layer / File(s) Summary
CI 동시성과 워크플로 검증
.github/workflows/ci.yml, .github/workflows/codeql.yml, services/account_unification/tests/test_workflow_concurrency_contract.py
pull_request에서는 PR 번호를, 그 외 이벤트에서는 run_id를 동시성 그룹 키에 사용합니다. 실행 중인 작업 취소는 pull_request에서만 활성화됩니다. 로컬 CodeQL 워크플로를 삭제하고 관련 계약 테스트를 추가했습니다.

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/actions-queue-consolidation

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 merged commit 100917d into main Sep 4, 2026
6 of 18 checks passed
@seonghobae
seonghobae deleted the codex/actions-queue-consolidation branch September 4, 2026 15:22
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