Code-Review on Jake Worth

On commenting and approving pull requests

Wed, 22 Apr 2026 11:06:20 -0400

After reviewing a lot of pull requests, I’ve settled on a simple default: if my comments are all nitpicks, suggestions, questions, or non-blocking issues, I leave them and approve the PR at the same time.

Things I Check Before Opening a PR

Sat, 07 Feb 2026 14:58:34 -0500

What is a programmer but a series of PRs (pull requests)? I optimize PRs to introduce the best code I can, be easy to review, and document my work so I can make sense of it in the future. Here are some things I always check before opening a PR.

How I Review Code, Part 2

Thu, 06 Jun 2024 00:00:00 +0000

Reviewing code is tricky. When I’m doing it, I’m trying to achieve a few things at once. In this post, I’d like to document the ways I try to add value via code reviews.

Why Great PRs Are Great: Pull Requests Templates

Mon, 14 Aug 2023 00:00:00 +0000

Have you ever seen a pull request that seems to completely explain itself? It’s a real artifact. I don’t know the project, yet I understand it. How can we get results like this on every pull request, from every developer on the team, every time?

Don't Ask for Advice; Ask for a Code Review

Wed, 09 Aug 2023 00:00:00 +0000

Here’s some advice about programming I’ve found useful: “Don’t ask for advice; ask for a code review.” In this post, I’d like to explore what I think this adage means.

How I Review Code

Sun, 06 Mar 2022 00:00:00 +0000

Code reviews are important on many teams. Do them well, and your code ships quickly and safely. Do them poorly, and your code ships slowly and riskily. I try to contribute good code reviews. In this post, I’ll share my process.

How and Why to Squash Your Pull Request

Sun, 03 Jul 2016 00:00:00 +0000

Many pull requests go through a cycle: programmer opens pull request, maintainer gives feedback, programmer makes changes, repeat until ready to merge, maintainer merges. Prior to the merge, the pull request can be messy, full of reverts, fixups, and WIP commits. In the end, those commits are noise. We can tell a better story by squashing the branch.