Post #3417441
2026-06-27 01:26 UTC
I used to get annoyed at engineers that rely on a saved search to figure out what to review next. But lately, I think I’ve come to understand the utility.
MRs need to pass certain quality gates before they’re ready for me to review (and hopefully merge). If those gates aren’t met, it’s a waste of my time to review.
1/2
Replies (1)
-
@ssemicolons@phpc.social 2026-06-27 01:28
Cont. If I check the whole queue 4x a day (or even 2x) to make sure CI is passing, SRP, SAST/Snyk passing, all the labels are right and accurate…I’d NEVER get anything done. The MR author HAS to be trusted to do those things. If they don’t, it doesn’t get reviewed, because it doesn’t get seen. And, the ones that ARE ready to review, usually sail right though. 2/2