top | item 43226147

(no title)

avikalp | 1 year ago

This is only true for "development branch" CRs/pull requests. The whole is greater than the sum of the parts. Every small change in the feature that you are building might make complete sense, so every dev-to-feature branch pull request would get approved easily.

But if you not also reviewing the feature-to-main branch pull request, you are just inviting problems. That is a bigger CR that you should review carefully, and there is no way that could be a small CR.

discuss

order

shermantanktop|1 year ago

I’ve done that gig many times. I take your point but if there’s a problem, figuring out who to blame always means going back to the original dev CR.