test(ci): prove declared Python 3.10 minimum runtime - #712
Closed
seonghobae wants to merge 1 commit into
Closed
seonghobae wants to merge 1 commit into
seonghobae wants to merge 1 commit into
Conversation
Contributor
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
Contributor
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fail-first slice for #623
Protected
mainat 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.pyrequires the existingpython-matrixjob to contain3.10,3.12, and3.14, retain fail-slow execution, install the same hash-locked CI dependencies, build/verify the compiled Rust-primary package, and run fullpytest. 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
pythoncheck, 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.