Skip to main content
Team Processes

Why Your Code Review Process Is Too Slow: The One-Day Rule

Google's one-business-day response rule is the metric your team is missing. Here's how to enforce it without burning out your reviewers.

Imagine you push a branch, open a pull request, and then wait. And wait. Forty-eight hours later, you finally get a comment: "Looks good, but can you move this function to the other file?" You fix it, push, and the cycle repeats. By the time your change merges, you've lost two days and all your context. If this sounds familiar, your code review process isn't just slow—it's broken. And the fix isn't a new tool or a longer checklist. It's a simple, ruthless rule: every review request gets a response within one business day.

The Real Cost of Slow Reviews

Slow reviews don't just annoy developers; they cost your team real momentum. When a change sits in review, it accumulates merge conflicts, stale context, and mental overhead. The developer who wrote it has moved on to other tasks, and every comment forces them to context-switch back. That's wasted time, and it compounds across every pull request in your queue.

Google has studied this problem at scale. In a 2018 case study of over 9 million reviewed changes, they found that the median time to first feedback for small changes was under one hour, and the overall median review latency was under four hours (Modern Code Review: A Case Study at Google). That's the benchmark. If your team is taking days to respond, you're not just slower than Google—you're slower than what's humanly reasonable.

The One-Day Rule Is Non-Negotiable

Google's engineering practices are explicit: a code review request should get a response within one business day. Not a full review, not a detailed analysis—a response. That means a reviewer should acknowledge the request, say "I'll get to it," or even just post a quick comment, before the next morning starts (Google Engineering Practices - Speed of Code Reviews).

Why so strict? Because the first response sets the tone. When a developer knows they'll hear back within 24 hours, they're more likely to keep the change fresh in their mind. They're more likely to make follow-up fixes quickly. And they're far less likely to abandon the change altogether and let it rot in the review queue.

But here's the kicker: the rule isn't just about speed for speed's sake. Google's research shows that quick responses reduce developer frustration more than a fast overall review time does (Google Engineering Practices - Speed of Code Reviews). That's a subtle but crucial point. A developer can tolerate a long, thorough review as long as they're getting feedback regularly. It's the silence that kills.

How to Actually Make It Happen

You can't just tell your reviewers "be faster" and expect results. You need structural changes. First, shrink your pull requests. Google's guidance is that 100 lines is a reasonable size for a change, and 1000 lines is usually too large (Google Engineering Practices - Small CLs). At Google, the median change size was about 24 lines (Modern Code Review). Think about that: half of all Google changes are smaller than a single screen of code.

Small changes are faster to review, more thoroughly reviewed, and less likely to introduce bugs (Google Engineering Practices - Small CLs). When you have a 24-line change, you can review it in minutes. When you have a 500-line monster, you're looking at an hour, and that's why reviewers procrastinate.

Second, embrace the "LGTM with comments" technique. Google reviewers are encouraged to approve a change even if they have minor comments, as long as they're confident the developer will address them (Google Engineering Practices - Speed of Code Reviews). This isn't about rubber-stamping; it's about unblocking the author. If your only feedback is "sort the imports" or "fix this typo," don't hold the merge hostage. Approve, leave the comment, and move on.

Third, automate the mechanical stuff. Linters, formatters, static analysis, and secret scanners should handle style and obvious bugs before a human ever looks at the code (Google Engineering Practices - code review). Tools like ESLint for JavaScript, Bandit for Python, and gosec for Go can catch common issues automatically (ESLint, Bandit, gosec). SonarQube can even enforce quality gates, like requiring no new issues and at least 80% test coverage on new code (SonarQube Server Docs). When the machine handles the boring stuff, the human reviewer can focus on what matters: design, logic, and architecture.

The Exception That Proves the Rule

Of course, there are changes that need more than a quick look. A 500-line architectural shift in a critical service deserves a careful, multi-hour review. But even then, the reviewer should still respond within one business day—acknowledging the change, setting expectations, maybe asking clarifying questions. The speed rule applies to the initial response, not the entire review.

And here's the thing: most changes don't need that level of scrutiny. Google's data shows that over 80% of reviews finish in a single iteration of resolving comments, and fewer than 25% of changes have more than one reviewer (Modern Code Review). That means the vast majority of reviews are straightforward. They don't need a week-long deliberation; they need a quick, focused pass from a competent engineer.

So my recommendation is simple: adopt the one-day rule as a team policy. Track your median time to first response. If it's over 24 hours, you have a problem. And don't just track it—enforce it. Set up branch protection rules that require reviews, but also set up a culture where reviewers are expected to respond quickly. If a reviewer consistently ignores requests, that's a performance issue, not a personality quirk.

Your developers will thank you. Your codebase will get healthier. And you'll stop losing days to the review queue.

The Takeaway

Slow code review is a process failure, not a tooling problem. The fix is to make speed a non-negotiable part of your team's culture. Adopt Google's one-business-day response rule. Keep your pull requests small—aim for 100 lines, not 1000. Use automation to handle the mechanical checks. And approve with comments when the feedback is minor. Your team will ship faster, and your reviewers will stop dreading the queue.

Sources

  • Google Engineering Practices - Code Review - https://google.github.io/eng-practices/review/
  • Google Engineering Practices - Speed of Code Reviews - https://google.github.io/eng-practices/review/reviewer/speed.html
  • Google Engineering Practices - Small CLs - https://google.github.io/eng-practices/review/developer/small-cls.html
  • Modern Code Review: A Case Study at Google - https://www.papercache.org/papers/mlsys/system/2026/03/25/modern-code-review-a-case-study-at-google
  • SonarQube Server Docs - Understanding quality gates - https://docs.sonarsource.com/sonarqube-server/2026.1/quality-standards-administration/managing-quality-gates/introduction-to-quality-gates

Share this article:

Comments (0)

No comments yet. Be the first to comment!