ci: remove temporary linux64_ubsan workaround following #7503 - #7554
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
WalkthroughThe CI matrix script no longer handles the Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
✅ Final review complete — no blockers (commit 1546b0b) |
thepastaclaw
left a comment
There was a problem hiding this comment.
Final validation — Codex + Sonnet
This is a trivial, purely mechanical CI cleanup that removes the temporary linux64_ubsan compatibility branch from ci/dash/matrix.sh, which was a documented workaround pending PR #7503's merge (already merged per the commit graph). Independently verified the diff, confirmed zero remaining references to linux64_ubsan anywhere in the repo (workflows now consistently use linux64_asan), and confirmed the script still executes cleanly. Both Codex and Sonnet reviewer lanes (general + commit-history) returned zero findings, and CodeRabbit posted no actionable inline comments — this assessment is unanimous and correct.
Source: reviewer backends codex-general gpt-5.6-sol, codex-dash-core-commit-history gpt-5.6-sol, sonnet-general claude-sonnet-5, sonnet-dash-core-commit-history claude-sonnet-5; final verifier backend claude-sonnet-5. openclaw-agent/cliproxy/gpt-5.6-sol is orchestration-only and is not reviewer evidence.
Review provenance
- Codex reviewers:
gpt-5.6-sol— general (completed),gpt-5.6-sol— dash-core-commit-history (completed) - Verifier:
claude-sonnet-5— final-verifier - Sonnet reviewers:
claude-sonnet-5— general (completed),claude-sonnet-5— dash-core-commit-history (completed)
Issue being fixed or feature implemented
Follow-up to PR #7503. Removes the temporary CI matrix workaround for
linux64_ubsanthat mappedlinux64_ubsanto00_setup_env_native_asan.shwhile testing #7503 before merge.What was done?
Removed the
elif [ "$BUILD_TARGET" = "linux64_ubsan" ]; thenworkaround block inci/dash/matrix.sh.How Has This Been Tested?
Ran
test/lint/lint-shell.pylocally.This pull request was created by Codex.
Breaking Changes
N/A
Checklist: