--- title: The Code Review Guide | daily.dev description: Code reviews are a common pratice in tech industry. When you do a Pull Request / Merge Request someone has to review it, give feedback, and approve it. You can... canonical: https://daily.dev/posts/the-code-review-guide-adkv14xx4 twitter:card: summary_large_image twitter:site: @dailydotdev og:type: website og:site_name: daily.dev og:title: The Code Review Guide | daily.dev og:description: Code reviews are a common pratice in tech industry. When you do a Pull Request / Merge Request someone has to review it, give feedback, and approve it. You can... og:url: https://daily.dev/posts/the-code-review-guide-adkv14xx4 og:image: https://api.daily.dev/og/posts/AdKV14xx4.png og:image:alt: The Code Review Guide 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. # The Code Review Guide **[DEV](https://daily.dev/sources/devto)** · 3 min read · 13 upvotes · 2 comments ## Summary Code reviews are a common pratice in tech industry. When you do a Pull Request / Merge Request someone has to review it, give feedback, and approve it. You can review it by two approaches: readability and functionality. As a reviewer you’re also responsible for the quality of the code that the company ships to production. ## 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. **@arklev** · 4 upvotes > i can relate to this, first time it does feel like you are attacking your teammate but after you under stand that its all for the greater good of the product. --- Tags: [#general-programming](https://daily.dev/tags/general-programming) [View this post on daily.dev](https://daily.dev/posts/the-code-review-guide-adkv14xx4) ```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":"The Code Review Guide","url":"https://daily.dev/posts/the-code-review-guide-adkv14xx4","mainEntityOfPage":{"@type":"WebPage","@id":"https://daily.dev/posts/the-code-review-guide-adkv14xx4"},"datePublished":"2020-10-07T00:01:12.136Z","dateModified":"2021-10-07T14:17:10.317Z","description":"Code reviews are a common pratice in tech industry. When you do a Pull Request / Merge Request someone has to review it, give feedback, and approve it. You can...","image":"https://media.daily.dev/image/upload/f_auto,q_auto/v1/posts/40310c4ee648287c6269c0a73614d5dd","thumbnailUrl":"https://media.daily.dev/image/upload/f_auto,q_auto/v1/posts/40310c4ee648287c6269c0a73614d5dd","isAccessibleForFree":true,"articleSection":"DEV","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":"DEV","logo":"https://media.daily.dev/image/upload/t_logo,f_auto/v1/logos/devto","url":"https://daily.dev/sources/devto"},"commentCount":2,"discussionUrl":"https://daily.dev/posts/the-code-review-guide-adkv14xx4","interactionStatistic":[{"@type":"InteractionCounter","interactionType":{"@type":"LikeAction"},"userInteractionCount":13},{"@type":"InteractionCounter","interactionType":{"@type":"CommentAction"},"userInteractionCount":2}],"keywords":"general-programming","timeRequired":"PT3M"} {"@context":"https://schema.org","@type":"BreadcrumbList","itemListElement":[{"@type":"ListItem","position":1,"name":"Home","item":"https://daily.dev"},{"@type":"ListItem","position":2,"name":"DEV","item":"https://daily.dev/sources/devto"},{"@type":"ListItem","position":3,"name":"The Code Review Guide"}]} {"@context":"https://schema.org","@type":"WebPage","@id":"https://daily.dev/posts/the-code-review-guide-adkv14xx4","comment":[{"@type":"Comment","text":"i can relate to this, first time it does feel like you are attacking your teammate but after you under stand that its all for the greater good of the product.","datePublished":"2020-10-07T07:56:41.237Z","url":"https://daily.dev/posts/AdKV14xx4#c-6zFcKB5Ry","author":{"@type":"Person","name":"Arkady Levin","url":"https://daily.dev/arklev","image":"https://avatars0.githubusercontent.com/u/12517285?v=4"},"interactionStatistic":{"@type":"InteractionCounter","interactionType":{"@type":"LikeAction"},"userInteractionCount":4}}]} ```