Skip to content

test(ci): prove declared Python 3.10 minimum runtime - #712

Closed
seonghobae wants to merge 1 commit into
mainfrom
test/python-310-minimum-runtime-red-623
Closed

seonghobae wants to merge 1 commit into
mainfrom
test/python-310-minimum-runtime-red-623

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Fail-first slice for #623

Protected main at branch creation: 56967deceafeb7ecd0ff08a98f196282a6ec81a3.

The package advertises requires-python = ">=3.10", while the accepted full-suite CI matrix currently exercises only CPython 3.12 and 3.14. This Draft adds only the smallest intentional RED proving the declared minimum runtime must execute the same Rust-primary full suite rather than remain metadata-only support.

tests/test_ci_python_310_contract.py requires the existing python-matrix job to contain 3.10, 3.12, and 3.14, retain fail-slow execution, install the same hash-locked CI dependencies, build/verify the compiled Rust-primary package, and run full pytest. Current protected main is expected to fail only the new matrix-membership assertion. Setup/import/fixture/dependency installation failure is not valid RED for this first commit.

After exact-head CI proves that source boundary, implement the smallest workflow change by adding CPython 3.10 to the existing matrix. Then let real CI determine whether the current hash-locked dependencies and code genuinely support 3.10. If the 3.10 lane exposes a real dependency/syntax/API incompatibility, fix the root cause without dropping hashes, skipping tests, xfail-ing compatibility, or narrowing the suite; narrow package metadata only through a separate evidence-backed compatibility decision if support is genuinely not intended/feasible.

Preserve Python 3.12/3.14, the aggregate required python check, Rust/PyO3/default-Rust verification, package/reinstall, GPU, fuzz, Security/SAST and immutable action pins. Add authoritative changelog/support-documentation updates only after GREEN. No psychometric numerical semantics, provider/dependency version bump, release, or canonical #604 documentation branch belongs here.

Keep Draft through RED→GREEN and current-head review/gates. Advances #623.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: f8acd22d-2cf4-4d16-a2f2-340a4c5b1d2f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

Copy link
Copy Markdown
Contributor Author

Superseded by requires-python >=3.12 floor (#703 restored via #713). Declared minimum is no longer 3.10.

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