Remove PR checklist bot workflow#36063
Conversation
Nobody reads the automated PR comment — it's noise. The same guidance (PR title conventions, test coverage, design docs, proto labels, cloud companion PRs) lives in doc/developer/guide-changes.md and is enforced through code review. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Thanks for opening this PR! Here are a few tips to help make the review process smooth for everyone. PR title guidelines
Pre-merge checklist
|
|
I always use the "trigger-ci for additional test/nightly runs" |
I also use that link a lot, is there maybe a good way we can add a button right in github (I know you can set up a bookmark to do that, but then everyone has to do that themselves...). But other than that I'd be in favor of removing the message! |
|
We could use a github status for that, it's slightly more hidden. Added a commit for it. Here's a demo of how it looks for Vercel: Screen.Recording.2026-04-15.at.14.34.03.mov |
9125cc6 to
15acd80
Compare
Neat! It seems github sorts the checks alphabetically, so we could pick a name that sorts earlier. |
|
Yes, I also just realized that, so using "A" now :D |
To reduce noise. Follow-up to MaterializeInc#36063
To reduce noise. Follow-up to MaterializeInc#36063
To reduce noise. Follow-up to MaterializeInc#36063

Nobody reads the automated PR comment — it's noise.
The same guidance (PR title conventions, test coverage, design docs, proto labels, cloud companion PRs) lives in
doc/developer/guide-changes.mdand is enforced through code review.🤖 Generated with Claude Code