Why is LGTM scary? We don't know if the reviewer reviewed it, spent time to understand the code change, knows what the code is supposed to do, etc. As a senior eng, sometimes I wanted to steer the review to be thorough, but inspection doesn't scale.....
LGTM - the scariest words to see as a CTO. That means devs didn't spend enough time on a PR.
Stacked PRs - they look like fun, but they're terrible. They unblock the author but they make code reviews a nightmare. They're a code review villain in disguise!
Bottlenecks in PRs are everywhere - but the core issue is lacking context. Devs & managers don't know how close a PR is to shipping. Reviewers don't know if major changes are coming or how the author got to the final code. They also lack a product th...
If a code review was done by AI, what would you lose out on? If an AI told you, before you request a review, that it's going to be rejected because of reasons XYZ, would that be better?
As a dev, I write a bunch of code. I just want to get a PR merged. I am often frustrated to know if a dev is even looking at my PR, how close they are to wrapping it up, etc. Would knowing this make you happier or more frustrated? 👍 = yes 👎 = no Re...