Repository navigation
fix(ci): key Python environment caches by resolved interpreter version - #134
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (2)
Included review availability: This review used your included allowance. 3 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughCI and publishing workflow cache keys now include runner OS and architecture. Several CI keys use the Python version reported by Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~4 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to The reviewed cache keys distinguish resolved Python versions where cached environments are reused, so no merge-blocking cache risk was identified. 🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
Comment |
|
📦 Python package built successfully!
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #134 +/- ##
=======================================
Coverage 77.26% 77.26%
=======================================
Files 115 115
Lines 6589 6589
Branches 961 961
=======================================
Hits 5091 5091
Misses 1186 1186
Partials 312 312
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. |
|
🚀 Review App Deployment Started
|
Runner image updates can restore virtual environments created with a different Python patch version, causing Poetry to recreate the environment and unit tests to fail with
Command not found: pytest.Key CI and CD environment caches by the resolved Python version, OS, and architecture. Dependency installation continues to be skipped on cache hits.
Validation: both workflow YAML files parse, cache step references checked, and
git diff --checkpassed.Summary by CodeRabbit