Large gap between opening a PR and reviews being submittedWe've noticed interesting trends...Tuesdays have the most code reviews. Wednesdays have the most PRs created. If we are supposed to get code reviewed fast, why is there such a large gap? (Note that not everyone is opening PRs right when they start wo...May 3, 2023·1 min read
LGTM - why it's scaryWhy 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.....Mar 28, 2023·1 min read
LGTM = Scary!LGTM - the scariest words to see as a CTO. That means devs didn't spend enough time on a PR.Mar 23, 2023·1 min read
Stacked PRs are bad!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!Mar 22, 2023·1 min read
Context in PRs is not being solvedBottlenecks 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...Mar 21, 2023·1 min read
Is AI bad for code reviews?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?Mar 18, 2023·1 min read
"How is the review coming along"...would this be useful?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...Mar 17, 2023·1 min read