Skip to content

ci: keep Crush reviews visible and lightweight#6

Merged
daltoniam merged 2 commits into
mainfrom
fix/crush-review-lightweight-status
Jun 7, 2026
Merged

ci: keep Crush reviews visible and lightweight#6
daltoniam merged 2 commits into
mainfrom
fix/crush-review-lightweight-status

Conversation

@daltoniam

Copy link
Copy Markdown
Owner

Summary

  • Use existing CI status during Crush reviews instead of rerunning cargo verification work.
  • Leave a visible fallback comment with CI status when GitHub blocks own-PR approvals.

Test plan

  • git diff --check
  • actionlint .github/workflows/crush-pr-review.yml

💘 Generated with Crush

Use CI status instead of rerunning verification during automated reviews and leave a fallback PR comment when GitHub blocks own-PR approvals.

💘 Generated with Crush

Assisted-by: Crush:gpt-5.5

Co-Authored-By: gpt-5.5 <noreply@anthropic.com>
@daltoniam

Copy link
Copy Markdown
Owner Author

Clean PR — CI/status checks are passing (build: SUCCESS) and I didn't find any blocking issues. LGTM. (Approval blocked by GitHub's own-PR restriction.)

Keep the required developer metadata while avoiding repo, tool, and task type values in Cloudflare AI Gateway logs.

💘 Generated with Crush

Assisted-by: Crush:gpt-5.5

Co-Authored-By: gpt-5.5 <noreply@anthropic.com>
@daltoniam

Copy link
Copy Markdown
Owner Author

CI/status checks: build succeeded. Crush PR Review in progress. Review found zero Must Fix or Should Fix items across all categories (ABI, Rust quality, security, manifest, tests). Clean lightweight CI change. LGTM.

@daltoniam daltoniam merged commit 51bcc27 into main Jun 7, 2026
2 checks passed
@daltoniam daltoniam deleted the fix/crush-review-lightweight-status branch June 7, 2026 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant