ci: unblock release parity and current-main warnings - #8851
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 8 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe PR records a Linux HTTP Agent lifecycle timeout as a known parity failure, updates the related changelog, and narrows imports in packed-loop index-store lowering modules. ChangesLinux parity tracking
Codegen import cleanup
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to The PR records a scoped Linux parity exception, removes reported unused imports, and updates release notes without changing executable compiler behavior; no actionable merge-blocking risk remains after normal checks and review. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. (1 skipped: 1 unsupported.) Full details: Description checkExplanation The description explains the parity failure, issue tracking, warning cleanup, changelog updates, evidence, validation commands, and side-effect status. It does not use the template headings or include the checklist, but the required information is mostly present.
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 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 |
What
test_issue_4975_http_agent_keep_alive_connection_optionsin the parity ratchet.bug-open, scoped to Linux, and track the lifecycle fix in release: pre-existing Linux parity failures block v0.5.1519 #8841.Evidence
r17 Full: https://github.com/PerryTS/perry/actions/runs/32935157745
Parity job: https://github.com/PerryTS/perry/actions/runs/32935157745/job/98076809981
Parity artifact: https://github.com/PerryTS/perry/actions/runs/32935157745/artifacts/9597975431
Node exits 0. Perry prints the expected
trueand1250, then does not terminate before the harness deadline. The ratchet will fail stale once the fixture passes and require this entry to be removed with the product fix.The first PR run also exposed ten unused-import errors already present on current
mainafter #8847: https://github.com/PerryTS/perry/actions/runs/32945577563/job/98105537416. The follow-up commit removes exactly those unused imports; executable compiler logic is unchanged.Local checks
cargo fmt --all -- --checkpython3 scripts/parity_known_failures.py --self-testpython3 scripts/parity_known_failures.py --auditjq empty test-parity/known_failures.jsongit diff --checkNo publisher was dispatched and no npm/tag side effects occurred.
Summary by CodeRabbit