DEV Community

Francesco Meli
Francesco Meli

Posted on

PR Reviews: The Silent Team Killer? Let's Talk About It ๐Ÿ”

Hey dev community! ๐Ÿ‘‹

I've been thinking about something that's been bugging me (and probably you, too): PR reviews. Not the technical part, I'm referring to the human aspect.

The Daily Drama We All Know

You know the drill:

  • PRs sitting for days while everyone assumes "someone else will review it"
  • @channel's messages flooding Slack: "Can someone please review my urgent PR?"
  • That one teammate who cranks out massive 500 line PRs that nobody wants to touch
  • The guilt when you realize you've been the bottleneck on three different PRs

Sound familiar? ๐Ÿ˜…

My Experiment: A Different Approach

Instead of more pings and notifications, I built a simple bot that sends one daily summary to our team's Slack channel every morning. We split our week into:

  • Monday, Wednesday, Friday: Async PR reviews - everyone finds time to review on their own schedule
  • Tuesday, Thursday: Sync PR reviews - we have a 1-hour meeting before lunch where PR owners demo their changes live, then a randomly assigned presenter shares their screen to walk through the code

The daily message looks like this:

PR Updates for our team
[repo-name] Feature: User authentication
Reviewable | Async | Author: @john | Presenter: @paul | Updated: 2025-08-05 | Files: 12

[repo-name] Fix: Critical bug in payment flow  
Ready to merge | Sync | Author: @sarah | Presenter: @mike | Files: 3
Enter fullscreen mode Exit fullscreen mode

Here's what each part means:

  • Async vs Sync: The PR owner decides whether to demo changes to everyone (sync) or let people review individually (async)
  • Random Presenter: For sync reviews, someone is randomly assigned to actively examine the code and ask questions - this ensures at least one person is deeply engaged with each PR
  • PR States:

    • Reviewable = No conflicts, ready for team feedback
    • Ready to merge = Has enough approvals, feedback for the owner
    • Conflicting = Needs attention before review can happen
    • Mergeable with conflicts = Approved but conflicts appeared after approval (someone else merged something)

File Counts: Shows total files changed, but separates out "noise" like tests and snapshots (configurable) so teams get a real sense of the change scope

What Happened Surprised Me

Instead of faster reviews, something more interesting happened: we developed a ritual.

The team started naturally checking this message every morning with coffee. People began budgeting time for reviews instead of treating them as interruptions. Large PRs became less common because the daily visibility made everyone conscious of their PR sizes.

It wasn't about speed; it was about sustainable habits.

The Questions I'm Wrestling With

This got me thinking about PR reviews differently:

Is the real problem coordination, or something deeper?

  • Are we treating symptoms (slow reviews) instead of causes (unclear priorities, poor slicing)?
  • Do teams need better tools, or better practices?
  • Is "faster" actually better, or do we need "more sustainable"?

What makes teams want to do reviews?

  • Is it social accountability? Clear expectations? Reducing cognitive load?
  • How do you balance thoroughness with timeliness?
  • Does team culture matter more than tooling?

Help Me Understand Your World

I'm curious about your team's experience:

๐Ÿค” Questions for you:

  1. What's your biggest PR review frustration? (Too slow? Too shallow? Wrong people reviewing?)
  2. How does your team currently coordinate reviews? (GitHub assignments? Slack chaos? Formal rotation?)
  3. What would make you excited to review code daily? (Better tooling? Different process? Team structure?)
  4. Have you found any approaches that work? (Tools, practices, cultural changes?)

The Bigger Picture

I'm wondering if there's space for tools that focus on team habits rather than individual productivity. Something that helps teams develop sustainable review practices instead of just automating the mechanics.

What do you think?

  • Is this a real problem worth solving?
  • Are there existing solutions that work well for your team?
  • Would you pay for something that helped build better review habits vs. just faster reviews?

Drop your thoughts below! Curious to hear how other teams handle this challenge.

PS: If you've built or used tools that address PR review coordination, I'd love to hear about them. Always looking to learn from what's working (or not working) out there.

Top comments (0)

Some comments may only be visible to logged-in visitors. Sign in to view all comments.