TE
TechEcho
Home24h TopNewestBestAskShowJobs
GitHubTwitter
Home

TechEcho

A tech news platform built with Next.js, providing global tech news and discussions.

GitHubTwitter

Home

HomeNewestBestAskShowJobs

Resources

HackerNews APIOriginal HackerNewsNext.js

© 2025 TechEcho. All rights reserved.

Ship Small Diffs (2017)

15 pointsby typingcyclistover 2 years ago

1 comment

juancnover 2 years ago
Well... unless it&#x27;s something really new, not a small change. Some changes are not amenable to small diffs.<p>In that case, you don&#x27;t just &quot;review the diff&quot;, diffs are <i>VERY</i> limited. Lots of context is lost (you still use it as a guide, but it&#x27;s not enough).<p>So, what to do?<p>Do a code walkthrough, involve reviewers early, if you&#x27;re on the reviewing end, check-out the thing and navigate the code with whatever tool will help you understand it best (IDEs, whatever).<p>Reviewing a large change should be closer to pair-programming than a classical code review.