Skip to content

ci: call the shared checks.yml instead of the local copy - #46

Closed
lesnik512 wants to merge 2 commits into
mainfrom
shared-checks
Closed

lesnik512 wants to merge 2 commits into
mainfrom
shared-checks

Conversation

@lesnik512

@lesnik512 lesnik512 commented Sep 15, 2026

Copy link
Copy Markdown
Member

Summary

Part of modern-python/.github#95. ci.yml and scheduled.yml call the shared checks.yml and the local _checks.yml goes. This is the first caller, chosen because it needs a service container: it exercises the workflow from the shared-checks branch ref in modern-python/.github before that PR merges, and proves the postgres inputs path.

Draft until modern-python/.github#101 merges; the ref then flips to @main.

Changes

  • Postgres comes from the service-* inputs, its password via -e in service-options; DB_DSN from test-env; DISABLE_SQLALCHEMY_CEXT_RUNTIME from free-threaded-env, so it applies on 3.14t only, as before.
  • The matrix is derived from requires-python now, so lint runs on 3.11 (the floor) where the local copy pinned 3.13, and pytest runs just install where the local copy ran uv sync --all-extras --no-install-project. Both are what the standard says; if either fails here it is a finding about this repo.
  • PYTHONDONTWRITEBYTECODE and PYTHONUNBUFFERED are dropped; they only affected log buffering.

Checklist

@lesnik512

Copy link
Copy Markdown
Member Author

Closed with modern-python/.github#95: the org is staying on per-repo _checks.yml.

@lesnik512 lesnik512 closed this Sep 15, 2026
@lesnik512
lesnik512 deleted the shared-checks branch September 15, 2026 16:28
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