Skip to content

ci: align with modern-di#3

Merged
lesnik512 merged 2 commits into
mainfrom
ci/align-with-modern-di
Jun 9, 2026
Merged

ci: align with modern-di#3
lesnik512 merged 2 commits into
mainfrom
ci/align-with-modern-di

Conversation

@lesnik512

Copy link
Copy Markdown
Member

Summary

Aligns this repo's CI with the modern-di reference pattern, matching the end-state of the recent lite-bootstrap migration.

  • Bump action pins: actions/checkout@v4@v6, extractions/setup-just@v2@v4, astral-sh/setup-uv@v3@v8.2.0.
  • Drop the codecov/codecov-action upload step. Coverage floor is still enforced locally via --cov-fail-under in pyproject.toml; codecov was dropped from modern-di itself and we follow.
  • Split ci.yml into a thin trigger wrapper + reusable _checks.yml (workflow_call). Sets the shape up to add more callers later without duplicating job definitions.

Resulting workflow files are byte-identical to lite-bootstrap's post-#115 state.

Test plan

  • PR CI green across the full Python matrix (3.10–3.14).
  • After merge, CODECOV_TOKEN repo secret can be deleted (Settings → Secrets and variables → Actions) — operator follow-up.

🤖 Generated with Claude Code

lesnik512 and others added 2 commits June 9, 2026 16:41
- Bump pins: checkout@v4 → @v6, setup-just@v2 → @v4, setup-uv@v3 → @v8.2.0
  across ci.yml and publish.yml.
- Drop codecov upload step from ci.yml. Coverage floor enforced by
  --cov-fail-under in pyproject.toml; modern-di also dropped codecov.
- Split ci.yml into thin trigger wrapper + reusable _checks.yml
  (workflow_call). Sets the shape up to add more callers later.

Resulting ci.yml/_checks.yml/publish.yml are byte-identical to
lite-bootstrap's post-PR #115 state.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The modern-di docs moved to GitHub Pages at modern-di.modern-python.org;
the stale RTD URL was missed during that migration. Matches the
companion modern-di-{fastapi,faststream,litestar} PRs in this campaign
(also bundled this fix).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@lesnik512 lesnik512 merged commit 453858a into main Jun 9, 2026
6 checks passed
@lesnik512 lesnik512 deleted the ci/align-with-modern-di branch June 9, 2026 14:24
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