--- title: How To Review Code | daily.dev description: A comprehensive guide to effective code reviewing based on two decades of experience. Emphasizes focusing on big picture design over syntax, the critical... canonical: https://daily.dev/posts/how-to-review-code-f5xymfz7v twitter:card: summary_large_image twitter:site: @dailydotdev og:type: website og:site_name: daily.dev og:title: How To Review Code | daily.dev og:description: A comprehensive guide to effective code reviewing based on two decades of experience. Emphasizes focusing on big picture design over syntax, the critical... og:url: https://daily.dev/posts/how-to-review-code-f5xymfz7v og:image: https://api.daily.dev/og/posts/f5XYmFz7V.png og:image:alt: How To Review Code og:image:width: 1200 og:image:height: 630 og:locale: en --- > ## Documentation Index > Fetch the complete documentation index at: https://daily.dev/llms.txt > Use this file to discover all available pages before exploring further. # How To Review Code **[Matthias Endler](https://daily.dev/sources/endler)** · 10 min read · 493 upvotes · 20 comments ## Summary A comprehensive guide to effective code reviewing based on two decades of experience. Emphasizes focusing on big picture design over syntax, the critical importance of good naming, being decisive when rejecting changes, and treating reviews as iterative communication processes. Key principles include running code locally when possible, asking clarifying questions, avoiding nitpicking on formatting, and continuously learning from the review process. ## Full article daily.dev links to this article rather than hosting it. Read it at the original source: ## Community discussion Top comments from developers on daily.dev. **@fnova** · 3 upvotes > Great! Thanks for sharing. **@codermatt** · 3 upvotes > Nice! Also positive feedback is really nice to receive and also to give. If there are issues in the PR then ask questions to open up discussions. This is better then saying this is wrong and this is the fix. **@hiboudev** · 1 upvotes > Thanks for this! Helped me a lot with my project that I'm working on! **@nowrica** · 0 upvotes > LGTM **@infyneis** · 0 upvotes > "SGTM 👌" > Most satisfying PR comment to read. ## Similar posts on daily.dev - [Don’t just attend KubeCon \+ CloudNativeCon, Merge Forward your experience\!](https://daily.dev/posts/don-t-just-attend-kubecon-cloudnativecon-merge-forward-your-experience--l0rpp73x8) · CNCF · 1 upvotes · 0 comments - [Announcing H2 2026 KCDs](https://daily.dev/posts/announcing-h2-2026-kcds-m96goajm1) · CNCF · 1 upvotes · 0 comments - [Two months of Open Community Groups](https://daily.dev/posts/two-months-of-open-community-groups-asf52zhbs) · CNCF · 0 upvotes · 0 comments - [CNCF Unveils Schedule for KubeCon \+ CloudNativeCon Europe 2026](https://daily.dev/posts/cncf-unveils-schedule-for-kubecon-cloudnativecon-europe-2026-ikhcoa5cb) · CNCF · 2 upvotes · 0 comments - [CNCF Debuts KubeCon \+ CloudNativeCon Japan 2026 Schedule](https://daily.dev/posts/cncf-debuts-kubecon-cloudnativecon-japan-2026-schedule-xp5pyudub) · CNCF · 1 upvotes · 0 comments --- Tags: [#leadership](https://daily.dev/tags/leadership), [#code-review](https://daily.dev/tags/code-review) [View this post on daily.dev](https://daily.dev/posts/how-to-review-code-f5xymfz7v) ```json {"@context":"https://schema.org","@graph":[{"@type":"Organization","@id":"https://daily.dev/#organization","name":"daily.dev","url":"https://daily.dev","logo":{"@type":"ImageObject","url":"https://daily.dev/apple-touch-icon.png","width":180,"height":180},"sameAs":["https://twitter.com/dailydotdev","https://github.com/dailydotdev","https://www.linkedin.com/company/daily-dev-ltd"]},{"@type":"WebSite","@id":"https://daily.dev/#website","url":"https://daily.dev","name":"daily.dev","publisher":{"@id":"https://daily.dev/#organization"},"potentialAction":{"@type":"SearchAction","target":{"@type":"EntryPoint","urlTemplate":"https://daily.dev/search?q={search_term_string}"},"query-input":"required name=search_term_string"}}]} {"@context":"https://schema.org","@type":"TechArticle","headline":"How To Review Code","url":"https://daily.dev/posts/how-to-review-code-f5xymfz7v","mainEntityOfPage":{"@type":"WebPage","@id":"https://daily.dev/posts/how-to-review-code-f5xymfz7v"},"datePublished":"2025-08-05T23:34:10.667Z","dateModified":"2025-08-05T23:34:32.143Z","description":"A comprehensive guide to effective code reviewing based on two decades of experience. Emphasizes focusing on big picture design over syntax, the critical...","image":"https://media.daily.dev/image/upload/f_auto,q_auto/v1/posts/04db44f0da19625e7a163dfb0bf33ad0?_a=AQAEulh","thumbnailUrl":"https://media.daily.dev/image/upload/f_auto,q_auto/v1/posts/04db44f0da19625e7a163dfb0bf33ad0?_a=AQAEulh","isAccessibleForFree":true,"articleSection":"Matthias Endler","inLanguage":"en","publisher":{"@type":"Organization","name":"daily.dev","url":"https://daily.dev","logo":{"@type":"ImageObject","url":"https://daily.dev/apple-touch-icon.png","width":180,"height":180}},"author":{"@type":"Organization","name":"Matthias Endler","logo":"https://media.daily.dev/image/upload/s--YE8mbc4j--/f_auto/v1726990211/logos/endler","url":"https://daily.dev/sources/endler"},"commentCount":20,"discussionUrl":"https://daily.dev/posts/how-to-review-code-f5xymfz7v","interactionStatistic":[{"@type":"InteractionCounter","interactionType":{"@type":"LikeAction"},"userInteractionCount":493},{"@type":"InteractionCounter","interactionType":{"@type":"CommentAction"},"userInteractionCount":20}],"keywords":"leadership,code-review","timeRequired":"PT10M"} {"@context":"https://schema.org","@type":"BreadcrumbList","itemListElement":[{"@type":"ListItem","position":1,"name":"Home","item":"https://daily.dev"},{"@type":"ListItem","position":2,"name":"Matthias Endler","item":"https://daily.dev/sources/endler"},{"@type":"ListItem","position":3,"name":"How To Review Code"}]} {"@context":"https://schema.org","@type":"WebPage","@id":"https://daily.dev/posts/how-to-review-code-f5xymfz7v","comment":[{"@type":"Comment","text":"Great! Thanks for sharing.","datePublished":"2025-08-06T08:47:56.728Z","url":"https://daily.dev/posts/f5XYmFz7V#c-snRLhWenI","author":{"@type":"Person","name":"Filippo Nova","url":"https://daily.dev/fnova","image":"https://media.daily.dev/image/upload/s--oclVLqIW--/f_auto/v1743236014/avatars/avatar_v0frfYYUY"},"interactionStatistic":{"@type":"InteractionCounter","interactionType":{"@type":"LikeAction"},"userInteractionCount":3}},{"@type":"Comment","text":"Nice! Also positive feedback is really nice to receive and also to give. If there are issues in the PR then ask questions to open up discussions. This is better then saying this is wrong and this is the fix.","datePublished":"2025-08-09T08:00:52.350Z","url":"https://daily.dev/posts/f5XYmFz7V#c-o2JkswLEe","author":{"@type":"Person","name":"Matthew","url":"https://daily.dev/codermatt","image":"https://media.daily.dev/image/upload/s--vpgFMWFg--/f_auto/v1734948053/avatars/avatar_TwkyTVk6hJPYzRFUZDlm1"},"interactionStatistic":{"@type":"InteractionCounter","interactionType":{"@type":"LikeAction"},"userInteractionCount":3}},{"@type":"Comment","text":"Thanks for this! Helped me a lot with my project that I’m working on!","datePublished":"2025-09-03T12:26:51.599Z","url":"https://daily.dev/posts/f5XYmFz7V#c-4qh0fNvma","author":{"@type":"Person","name":"HibouDev","url":"https://daily.dev/hiboudev","image":"https://avatars.githubusercontent.com/u/180775069?v=4"},"interactionStatistic":{"@type":"InteractionCounter","interactionType":{"@type":"LikeAction"},"userInteractionCount":1}},{"@type":"Comment","text":"LGTM","datePublished":"2025-08-21T05:10:55.077Z","url":"https://daily.dev/posts/f5XYmFz7V#c-yeknGEGqg","author":{"@type":"Person","name":"하늘","url":"https://daily.dev/nowrica","image":"https://lh3.googleusercontent.com/a/ACg8ocJ0vF3RpmqvjBib4Hu6saQdN2xXX0XCRIhY7AqBsLnIez7oyona=s96-c"}},{"@type":"Comment","text":"“SGTM 👌”\nMost satisfying PR comment to read.","datePublished":"2025-08-11T13:23:41.664Z","url":"https://daily.dev/posts/f5XYmFz7V#c-2KkYbp1Cz","author":{"@type":"Person","name":"Samy","url":"https://daily.dev/infyneis","image":"https://media.daily.dev/image/upload/s--O39PTxJa--/f_auto/v1750074734/avatars/avatar_7z3D8ave6xqVOCXFFWxJC?_a=BAMClqUq0"}}]} ```