Skip to content

ci: split ci.yml into reusable _checks.yml#115

Merged
lesnik512 merged 1 commit into
mainfrom
ci/split-into-checks
Jun 9, 2026
Merged

ci: split ci.yml into reusable _checks.yml#115
lesnik512 merged 1 commit into
mainfrom
ci/split-into-checks

Conversation

@lesnik512

Copy link
Copy Markdown
Member

Summary

Structural mirror of modern-di's CI shape:

  • ci.yml becomes a thin trigger wrapper (push to main, pull_request, concurrency group) that delegates to _checks.yml.
  • _checks.yml is a reusable workflow (on: workflow_call) holding the lint and pytest jobs.

Both files are now byte-identical to their modern-di counterparts:

diff .github/workflows/ci.yml      ../modern-di/.github/workflows/ci.yml      # empty
diff .github/workflows/_checks.yml ../modern-di/.github/workflows/_checks.yml # empty

Why

  • Sets the shape up to add more callers later (nightly schedule, manual dispatch, dependent workflows) without duplicating the job definitions.
  • Aligns both repos in the modern-python org on one CI shape — easier to keep them in sync going forward.

Behaviour change

None. Same triggers, same concurrency group, same jobs, same steps, same matrix. The split is purely a workflow_call indirection.

Test plan

  • CI green on this PR — both the lint and pytest (5-Python matrix) reusable jobs should run via the checks caller in ci.yml.

🤖 Generated with Claude Code

Mirror modern-di's structural split: ci.yml is now a thin trigger wrapper
(push to main, pull_request, concurrency group) that calls _checks.yml
via workflow_call. _checks.yml holds the actual lint and pytest jobs.

Both files are byte-identical to their modern-di counterparts (no
divergence in steps, action versions, matrix, or caching). Sets the
shape up to add more callers later (e.g., a nightly schedule) without
duplicating job definitions.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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