Code Review Without the Hype

Abstract code gradient artwork

The fastest query is the one you never send, but the second fastest has a good index. Our on-call rotation improved the day we deleted half of the alerts. If the error message does not say what to do next, it is half an error message.

Getting started

The fastest query is the one you never send, but the second fastest has a good index. Tests are documentation that fails loudly when it goes stale. Naming things well saved us more time than any framework migration.

If the error message does not say what to do next, it is half an error message. We wrote the migration twice: once to learn, once to ship.

Context

Feature flags are debt with an expiration date you must actually enforce. The fastest query is the one you never send, but the second fastest has a good index. We measured first, and the bottleneck was not where anyone expected.

  • Write the failure mode down before writing code
  • Delete dead code the same week
  • Measure p95 and p99, not just the mean
  • Pin versions, update on purpose

Latency percentiles tell a story that averages politely hide. The best architecture diagram is the one the new hire can redraw from memory.

What worked

Tests are documentation that fails loudly when it goes stale. The rollback plan should exist before the rollout plan is approved. Our on-call rotation improved the day we deleted half of the alerts.

Latency percentiles tell a story that averages politely hide. The best architecture diagram is the one the new hire can redraw from memory.

Naming things well saved us more time than any framework migration. If the error message does not say what to do next, it is half an error message.

Comments

3 responses to “Code Review Without the Hype”

  1. Hugo B. Avatar
    Hugo B.

    Saving this for later, very practical.

  2. I tried this last week and can confirm it works.

  3. Priya S. Avatar
    Priya S.

    The photos and examples make this so much clearer.

Leave a Reply

Your email address will not be published. Required fields are marked *