Repository navigation
docs(conformance): record cachekit-py's path-encoding 1.2.0 and freshness-headers 1.0.0 copies (LAB-8254) - #160
Conversation
…ness-headers 1.0.0 copies
… the rendering case borrows
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 15 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (4)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (4)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. Summary by CodeRabbit
WalkthroughThe conformance records now identify updated Python fixture versions and hashes, revised API coverage statuses, and the related CI and release details. ChangesPython fixture conformance
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~8 minutes Change: Other Merge Risk: ⚪ Minimal · up to The recorded fixture versions and derived coverage agree, with no identified merge-blocking issue in this profile update. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 3 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Providing Context (Files & MCPs)Add these hints in your PR description (or a comment) to unlock deeper checks:
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
…54-merge # Conflicts: # conformance/coverage.md # sdk-feature-matrix.md
|
@kody review --force |
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Providing Context (Files & MCPs)Add these hints in your PR description (or a comment) to unlock deeper checks:
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
…54-merge2 # Conflicts: # conformance/coverage.md # sdk-feature-matrix.md
c4b817b
|
@kody review --force |
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Providing Context (Files & MCPs)Add these hints in your PR description (or a comment) to unlock deeper checks:
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
Records cachekit-py's byte-identical copies of
path-encoding.json1.2.0 andfreshness-headers.json1.0.0, vendored and CI-executed in cachekit-py#522.conformance/sdks.jsonnow reads cachekit-py atce427a2450bdb83a07e30d5b32449c556be1ff00, the merge commit of that PR onmain. Both new sha256 values were taken fromsha256sumof the copies in cachekit-py'stests/unit/protocol/fixtures/at that commit:path-encoding.json(1.2.0)807af0d39ccf3f5afda65577787ac5adcd01ff6617468060161c84267fa7e0cbfreshness-headers.json(1.0.0)74beb975b6855f52d9dd453279873faf8048a9c3c79a8167cfda099beec5fcd3The other five pinned cachekit-py copies hash the same at the new commit.
conformance/coverage.mdis regenerated. For cachekit-py, API-4 to API-7 and API-17 move frompartial (n/m)topartial (gap), and API-18, API-30, API-33, API-34 and API-57 move fromuncoveredtopartial (gap). cachekit-py's copies now hold every vector those ids map to. They stay belowcoveredbecause each id records a gap that no vector reaches. The cachekit-py totals move from 68 partial / 20 uncovered to 73 / 15.sdk-feature-matrix.md: the Test vectors in CI cell for cachekit-py lists both fixtures, and note ¹⁶ says cachekit-py vendors this repo's currentpath-encoding.jsonand is the only SDK to vendorfreshness-headers.json. The same cell called cachekit-py#391 and cachekit-py#397 unreleased, but both have shipped since PyPI 0.21.0 (in thev0.21.0tag), so the cell now says that.tools/test_conformance.py: one rendering case borrows threepath-encoding.json1.2.0 rows and expected every SDK to hold none of them. cachekit-py now holds them, so its column readspartial (gap). The case tests how names render, and its status expectation was the only part that changed.Checked locally:
python3 tools/conformance.py check --base origin/main(OK),tools/test_conformance.py(201/201),tools/test_changelog_collect.py,tools/check-version-floors.py.Closes LAB-8254