# A Guide for Team Leads on Code Review Best Practices | LinearB Blog

> Have you heard the old saying, “You can have something fast, cheap or good, but you can only pick two”? Software development is no exception. Code review best practices help you maintain a delicate balance between the need to be efficient and the need to be effective.  Table of Contents Git On The Same Page […]

_This is a markdown rendering of a live HTML page on linearb.io, generated for AI/LLM consumption — it is not a markdown-only site. To get the full HTML page instead, request this URL with an explicit `Accept: text/html` header (no wildcard, no markdown preference)._

[Blog](https://linearb.io/blog)

/

A Guide for Team Leads on Code Review Best Practices

# A Guide for Team Leads on Code Review Best Practices

![Photo of Nakia](https://assets.linearb.io/image/upload/c_limit,w_2560/f_auto/q_auto/v1/logo-mark-lg?_a=BAVMn6ID0)

By [Nakia](https://linearb.io/blog/code-review-best-practices#Nakia)

|

March 10, 2022

![pexels_christina_morillo_1181472_e1646959820869_6e32c33e20](https://assets.linearb.io/image/upload/c_limit,w_2560/f_auto/q_auto/v1/pexels_christina_morillo_1181472_e1646959820869_6e32c33e20?_a=BAVMn6ID0)

Have you heard the old saying, “You can have something fast, cheap or good, but you can only pick two”?

Software development is no exception. Code review best practices help you maintain a delicate balance between the need to be efficient and the need to be effective. 

## Table of Contents

* [Git On The Same Page with Code Review Goals](#git-on-the-same-page-with-code-review-goals)
* [Eliminate Idle Pull Requests](#eliminate-idle-pull-requests)
* [Limit PR Size](#limit-pr-size)
* [Improve Review Depth](#improve-review-depth)
* [Reinforce Code Review Best Practices](#reinforce-code-review-best-practices)
* [Celebrate Your Success](#celebrate-your-success)

## Git On The Same Page with Code Review Goals

What are your goals for improving your team’s code review process? Maybe you have some [code review best practices](https://linearb.io/blog/how-to-do-code-reviews-better/) documented already. Or maybe you need to canvas your team about what best practices they’re already using. Or you need to get [some new ideas](https://linearb.io/blog/pull-request-best-practices-our-tips/). 

It may come as a surprise to you, but some teams don’t use pull requests in their Git system at all. The downside to this is that it’s hard to know if your team is improving if you’re not tracking activity. Also, this may mean there is limited feedback from colleagues on code standards. And not to forget – this leads to more bugs being introduced into production, which creates problems for the user and for you, forcing you to put out the fire by hot fixing, rolling back to an older version, or [reverting the changes](https://linearb.io/blog/easily-revert-git-pull-requests/). If you’re already using pull requests, you’re one step ahead; if you’re not, keep reading to see the advantages.

Using PRs in Git and setting goals around them needs to be the first step in establishing best practices for your code reviews. Everybody doesn’t have to do things the same way, but when you set goals for your team, they’ll know what you expect from them and what to expect from their peers. Only then can your team can work with each other more efficiently. 

Let’s review 3 goals you can set with your team to improve their peer review process.

### Eliminate Idle Pull Requests 

It doesn’t do any good to use PRs in your Git if you don’t set goals around how quickly they’re reviewed. One of the most common DORA metrics is cycle time, but [cycle time is best understood in four phases](https://linearb.io/blog/how-to-calculate-cycle-time/), the second of which is pickup time.

![cycle time benchmarks](https://assets.linearb.io/uploads/cycle-time-benchmarks-1024x819.png)

Unfortunately, for most teams the average pickup time for pull requests is[ two days](https://linearb.io/blog/pull-request-pickup-time/). Quick: what did you have for lunch two days ago? If it took you a minute to remember, or if you don’t remember, then how can you expect a developer to remember the context of a pull request they submitted two days ago?

Eliminating wasted time between issuing a PR and starting to review code cuts down on the time your developers spend trying to remember and then explain. Our Data Science team evaluated nearly 2,000 dev teams and determined that [elite development teams have a PR pickup time of less than 7 hours](https://linearb.io/blog/engineering-metrics-benchmarks-what-makes-elite-teams/).

Pull requests gather dust for a few reasons. 

Sometimes reviewers don’t know the PR exists. Did the developer forget to assign a reviewer? Did they forget to email the team informing them of the need for a review? Did everyone think someone else was going to jump on it? Or did a reviewer get assigned, but they’re too busy with their own coding right now? We’re all human. These things happen.

Or maybe it wasn’t a mistake at all. Often the size or lack of context around the review can leave developers deciding to do almost anything else, which leads me to the next goal.

### Limit PR Size

A [Smartbear study](https://smartbear.com/learn/code-review/best-practices-for-peer-code-review/) revealed that reviewers become less likely to catch defects when they’re reviewing more than than 400 LOC in one sitting. This is a great start, but again, our [Data Science team](https://linearb.io/blog/the-pull-request-paradox-merge-faster-by-promoting-your-pr/) suggests you set your team goals even smaller if you want a truly elite team. Limit your PR size to no more than 225 changes. 

We found that not only do smaller pull requests lead to improved code quality but they also get merged faster. In contrast, large PR Sizes correlate to longer cycle time and code complexity. 

![pull request size](https://assets.linearb.io/uploads/pr-size.png)

### Improve Review Depth

Everybody can and should participate in reviewing code. As a team lead, maybe you don’t code or even review code anymore, but if you’re one of the most experienced team members, they still need your feedback in this capacity. 

One of the benefits of code reviews is the ability to level-up the experience of your team and [quality of their code](https://linearb.io/blog/5-key-metrics-to-fix-your-software-teams-quality/) quickly. But if your reviewers have too much WIP, lack context or are overwhelmed by the size of the review, you may end up with superficial, “looks good to me” comments just to merge faster.

![Review depth](https://assets.linearb.io/uploads/Frame-2182-1.png)

You can lead the team to improve in this area by setting and tracking goals around review depth. At LinearB, we calculate Review Depth by counting the number of comments on PRs throughout the iteration and dividing them by the total number of PRs submitted. Elite teams average more than 6 comments per review.

## Reinforce Code Review Best Practices

If a [pull request falls in the forest of GitHub](https://linearb.io/blog/code-review-on-github/), it doesn’t make a sound, but maybe it should. Setting goals and tracking metrics might improve your code review process a bit, but it won’t establish sustainable best practices in a meaningful way. Neither will harping on your devs meeting after meeting.

This is why we developed WorkerB, an automated bot that saves time. 

WorkerB can provide personal notifications to the developer and assigned reviewer via Slack or MS Teams. Plus, because we correlate your Git and project management data, like Jira, your reviewer gets context around which ticket the review is related to. 

We also give them an estimated review time, so they can schedule it around other work in progress and avoid context switching. Our personal alerts also feature a snooze button in case now isn’t the best time, but you don’t want to forget. 

Plus you can configure notifications that warn your team when your goals aren’t being met. Now the whole team is alerted and accountable for jumping in to give a basic review another look over before merging. Check it out.

[![lightning pull requests](https://assets.linearb.io/uploads/Lightning-Pull-Request-1024x497.png)](https://linearb.io/get-started/)

Avoid superficial LGTM reviews. Use Team Goals + WorkerB to reinforce good habits around review depth. [Get started today!](https://linearb.io/get-started/)

## Celebrate Your Success

Once you have team buy-in on your goals and you’re automating reinforcement of the best practices needed to be an elite dev team, you have to celebrate your success. Our Team Goals dashboard makes it easy to visual continuous improvement over time. 

[![team code review goals](https://assets.linearb.io/uploads/Group-6412_3x-1024x951.png)](https://linearb.io/demo/)

Track progress by sprint, week or month so you can celebrate team improvement regularly. Let your team have cake! [Book a demo of LinearB today.](https://linearb.io/demo/)

## Improve developer productivity with LinearB

Find us on

[](https://www.linkedin.com/company/linearb)
[](https://devinterrupted.substack.com/)

## Your next read

[![Cover image for Slack turns channels into the context engine for agentic AI](https://assets.linearb.io/image/upload/c_limit,w_2560/f_auto/q_auto/v1/Blog_Post_Name_2400x1256_077524ba8b?_a=BAVMn6ID0)](https://linearb.io/blog/slack-jaime-delanghe-mcp-agent-context-channels)

Workflow

[Slack turns channels into the context engine for agentic AI](https://linearb.io/blog/slack-jaime-delanghe-mcp-agent-context-channels)

Slack Chief Product Officer Jaime DeLanghe breaks down how channels serve as the foundational context layer for human-agent collaboration. Learn why Slack is...

[![Cover image for Code generation is faster than ever, but shipping value isn't](https://assets.linearb.io/image/upload/c_limit,w_2560/f_auto/q_auto/v1/Blog_AI_code_review_bottleneck_2400x1256_d73333ea46?_a=BAVMn6ID0)](https://linearb.io/blog/code-generation-faster-shipping-isnt)

Workflow

[Code generation is faster than ever, but shipping value isn't](https://linearb.io/blog/code-generation-faster-shipping-isnt)

Your team writes more code than ever, but less reaches production. AI moved the bottleneck to code review. Here's how to measure it and unblock your pipeline.&n...

[![Cover image for AI as a value multiplier: a human-centric approach to engineering leadership](https://assets.linearb.io/image/upload/c_limit,w_2560/f_auto/q_auto/v1/Blog_Servant_Leadership_2400x1256_0cfd4e2a0c?_a=BAVMn6ID0)](https://linearb.io/blog/ai-as-value-multiplier-human-centric-leadership)

Workflow

[AI as a value multiplier: a human-centric approach to engineering leadership](https://linearb.io/blog/ai-as-value-multiplier-human-centric-leadership)

Super.com's Matt Culver explains why AI should be used as a value multiplier, not a cost-cutter, advocating for a human-centric approach to engineering...

## Structured data

_Machine-readable metadata (JSON-LD) embedded in the page for search/AI context — not content rendered on the page itself._

```json
{
  "@context": "https://schema.org",
  "@type": "Organization",
  "name": "LinearB",
  "url": "https://linearb.io",
  "logo": "https://assets.linearb.io/image/upload/v1715628027/logo-mark-lg.svg",
  "description": "LinearB is the engineering productivity platform that helps engineering leaders prove AI is improving throughput without sacrificing delivery confidence, flow efficiency, or developer experience.",
  "sameAs": [
    "https://www.linkedin.com/company/linearb"
  ],
  "award": [
    {
      "@type": "Award",
      "name": "LinearB is a Leader in the 2026 Gartner® Magic Quadrant™ for Developer Productivity Insight Platforms",
      "dateAwarded": "2026",
      "awardedBy": {
        "@type": "Organization",
        "name": "Gartner®"
      }
    },
    {
      "@type": "Award",
      "name": "Great Place to Work Certification",
      "dateAwarded": "2025-2027",
      "awardedBy": {
        "@type": "Organization",
        "name": "Great Place to Work"
      }
    },
    {
      "@type": "Award",
      "name": "America's Best Startup Employers 2025",
      "dateAwarded": "2025",
      "awardedBy": {
        "@type": "Organization",
        "name": "Forbes Magazine"
      }
    }
  ],
  "hasCertification": [
    {
      "@type": "Certification",
      "name": "SOC 1 Type 2"
    },
    {
      "@type": "Certification",
      "name": "SOC 2 Type 2"
    },
    {
      "@type": "Certification",
      "name": "GDPR Compliance certification"
    },
    {
      "@type": "Certification",
      "name": "ISO 27001"
    }
  ]
}
```

```json
{
  "@context": "https://schema.org",
  "@type": "Article",
  "headline": "A Guide for Team Leads on Code Review Best Practices",
  "url": "https://linearb.io/blog/code-review-best-practices",
  "author": {
    "@type": "Person",
    "name": "Nakia"
  },
  "datePublished": "2022-03-10T18:50:58.000Z",
  "dateModified": "2022-03-10T18:50:58.000Z",
  "image": "https://assets.linearb.io/image/upload/v1720000000/pexels_christina_morillo_1181472_e1646959820869_6e32c33e20.jpg",
  "publisher": {
    "@type": "Organization",
    "name": "LinearB",
    "logo": "https://assets.linearb.io/image/upload/v1777485755/linearb-logo-2026.png"
  },
  "description": "Have you heard the old saying, “You can have something fast, cheap or good, but you can only pick two”? Software development is no exception. Code review best practices help you maintain a delicate balance between the need to be efficient and the need to be effective.  Table of Contents Git On The Same Page […]\n"
}
```

## More on linearb.io

### Top navigation

- [Book a Demo](https://linearb.io/book-a-demo)
- [AI Code Reviews — Catch security risks, bugs, and spec mismatches](https://linearb.io/platform/ai-code-reviews)
- [AI & Productivity Insights — See how AI tools affect cycle time and delivery speed](https://linearb.io/platform/ai-developer-productivity-insights)
- [Measure AI Impact — Track AI adoption and tie it to delivery outcomes](https://linearb.io/use-case/measure-ai-impact)
- [MCP Server — Chat with your data to spot patterns and boost output](https://linearb.io/platform/mcp-server)
- [Resource Allocation — Cost initiatives and shape your investment strategy](https://linearb.io/platform/resource-allocation)
- [Cost Capitalization — Capitalize engineering costs with audit-ready reports](https://linearb.io/platform/cost-capitalization)
- [Dev Team Management — Set targets and tie throughput to business outcomes](https://linearb.io/platform/goals-and-reporting)
- [DevOps Workflow Automation — Policy-based PR routing, approvals, and tests](https://linearb.io/platform/ai-workflow-governance)
- [AI Powered Support — Unify AI and human code delivery in one clear view](https://linearb.io/use-case/ai-powered-support)
- [Optimization — Surface friction with feedback and MCP insights](https://linearb.io/platform/developer-experience)
- [Reporting — Spot what's working and what needs attention](https://linearb.io/use-case/measuring-developer-experience)
- [Surveys — Turn developer feedback into actionable signals](https://linearb.io/platform/developer-surveys)
- [Platform overview](https://linearb.io/platform/overview)
- [Register now](https://linearb.io/event/engineering-productivity-gap)
- [Customers](https://linearb.io/customers)
- [Pricing](https://linearb.io/pricing)
- [Why choose LinearB — Explore your data. Measure performance. Act to improve it.](https://linearb.io/why-linearb)
- [APEX framework — The operating model for AI-era engineering teams](https://linearb.io/resources/apex-framework)
- [Anti-FAQ — The questions other vendors won't answer](https://linearb.io/why-linearb/anti-faq)
- [Security — Enterprise-grade compliance and zero code access](https://linearb.io/security)
- [Build vs. buy — The hidden cost of building it yourself](https://linearb.io/resources/build-vs-buy)
- [Dev Interrupted Podcast — Conversations with engineering leaders](https://linearb.io/dev-interrupted/podcasts)
- [Reports & Guides — Deep dives on productivity and delivery](https://linearb.io/resources)
- [Webinars — Expert sessions on productivity and AI](https://linearb.io/resources?category=workshops)
- [Metrics Benchmarks — See how your engineering org stacks up](https://linearb.io/resources/software-engineering-benchmarks-report)
- [Blog — Product updates and practical insights](https://linearb.io/blog)
- [Help Center — Documentation, setup, and support](https://linearb.helpdocs.io)
- [API Docs](https://docs.linearb.io/api-overview)
- [Status](https://www.linearbstatus.com/)
- [Integrations](https://linearb.io/integrations)
- [LinearB is a Leader in the 2026 Gartner® Magic Quadrant™ for Developer Productivity Insight Platforms](https://linearb.io/resources/gartner-magic-quadrant-dpi-platforms-2026)
- [Sign in](https://app.linearb.io/login)
- [Enterprise](https://linearb.io/solutions/enterprise)
- [Contact](https://linearb.io/contact-us)
- [About us](https://linearb.io/about-us)
- [Careers](https://linearb.io/careers)
- [Service agreement](https://linearb.io/services-agreement)
- [Privacy policy](https://linearb.io/privacy-policy)
- [DPA](https://linearb.io/data-processing-agreement)
- [Security FAQ](https://linearb.io/security-faq)
- [Substack](https://devinterrupted.substack.com/)

### Footer

_Additional links from the site footer, not repeated from the top navigation above._

- [GitHub](https://github.com/linear-b)
- [LinkedIn](https://www.linkedin.com/company/linearb)
- [Twitter](https://twitter.com/LinearB_Inc)