liveBook · Manning

Single senior developer reviewer problem

As explained in "Looks Good to Me"

A code-review workflow problem in which one senior developer is expected to review every pull request.

A team creates a review bottleneck when every pull request must be approved by its only senior developer. Seniority may have become a habit, an authority signal, or an automatic assignment rather than a genuine requirement. section 8.1

A healthier default is that anyone on the development team can approve a pull request, with junior and new developers’ approvals carrying the same weight. Shared approval depends on safeguards, not on treating one person as the final safety net. section 8.1

Build confidence with complementary safeguards:

  • Use multiple deployment environments or stages. section 8.1
  • Make deployed changes easy to revert. section 8.1
  • Run thorough CI checks, unit tests, and integration tests. section 8.1
  • Share accountability and maintain a blameless culture. section 8.1

How "Looks Good to Me" covers it

How deep it goes
Defines the bottleneck, recommends team-wide approval, and lists operational safeguards without prescribing implementation details.
Examples
Uses the recurring case of one senior developer receiving every pull request and contrasts it with team-wide approval and safeguards.
What it assumes
Assumes a development team with pull requests and identifies safeguards needed before approval responsibility is distributed.
What you can do after
Apply team-wide approval while adding automated checks, tests, staged deployment, reversibility, and shared accountability.
How it connects
Connects code review with pull requests, continuous integration, automated testing, deployment environments, rollback, accountability, and blameless culture.

Related concepts

This explainer was generated by Manning's knowledge graph from the text of "Looks Good to Me". The section links point to the parts of the book it draws on.