<?xml version="1.0" encoding="UTF-8"?>
<rss version="2.0"
	xmlns:content="http://purl.org/rss/1.0/modules/content/"
	xmlns:wfw="http://wellformedweb.org/CommentAPI/"
	xmlns:dc="http://purl.org/dc/elements/1.1/"
	xmlns:atom="http://www.w3.org/2005/Atom"
	xmlns:sy="http://purl.org/rss/1.0/modules/syndication/"
	xmlns:slash="http://purl.org/rss/1.0/modules/slash/"
	>

<channel>
	<title>To Hell in a Handbasket &#187; Peer Code Review</title>
	<atom:link href="http://ericsilva.org/tag/peer-code-review/feed/" rel="self" type="application/rss+xml" />
	<link>http://ericsilva.org</link>
	<description>General Views, Opinions, and Observations in the World Around Me by Eric Silva</description>
	<lastBuildDate>Mon, 05 Sep 2011 13:48:09 +0000</lastBuildDate>
	<language>en</language>
	<sy:updatePeriod>hourly</sy:updatePeriod>
	<sy:updateFrequency>1</sy:updateFrequency>
	<generator>http://wordpress.org/?v=3.3.1</generator>
		<item>
		<title>Code Review Does Not Have to Suck</title>
		<link>http://ericsilva.org/2011/02/24/code-review-does-not-have-to-suck/</link>
		<comments>http://ericsilva.org/2011/02/24/code-review-does-not-have-to-suck/#comments</comments>
		<pubDate>Thu, 24 Feb 2011 17:49:12 +0000</pubDate>
		<dc:creator>Eric Silva</dc:creator>
				<category><![CDATA[Computers]]></category>
		<category><![CDATA[Programming]]></category>
		<category><![CDATA[CodeCollaborator]]></category>
		<category><![CDATA[Crucible]]></category>
		<category><![CDATA[Kiln]]></category>
		<category><![CDATA[Peer Code Review]]></category>
		<category><![CDATA[SmartBear]]></category>

		<guid isPermaLink="false">http://ericsilva.org/?p=453</guid>
		<description><![CDATA[I have been using this saying for the last 6 years, &#8220;Code Review Does Not Have to Suck&#8221;.  I just saw today that SmartBear, who makes the wonderful CodeCollaborator peer code review tool, which I also use, released a quick 3-minute video explaining this. Peer code review is a critical milestone step in any good [...]]]></description>
			<content:encoded><![CDATA[<p>I have been using this saying for the last 6 years, &#8220;Code Review Does Not Have to Suck&#8221;.  I just saw today that SmartBear, who makes the wonderful <a href="http://smartbear.com/products/development-tools/code-review/features/">CodeCollaborator</a> peer code review tool, which I also use, released <a href="http://smartbear.com/support/screencasts/codecollaborator/not-suck-ad">a quick 3-minute video</a> explaining this.</p>
<p>Peer code review is a critical milestone step in any good development process; especially with distributed teams.  <a href="http://smartbear.com/products/development-tools/code-review/features/">CodeCollaborator</a>, <a href="http://www.atlassian.com/software/crucible/">Crucible</a>, and <a href="http://www.fogcreek.com/kiln/">Kiln</a> are all great tools for facilitating peer code review without having to go through all the headache of gathering files, scheduling meetings, pissing off your co-workers, etc.</p>
<p>Check out the <a href="http://smartbear.com/support/screencasts/codecollaborator/not-suck-ad">video</a>.  It&#8217;s worth the 3-minutes.</p>
]]></content:encoded>
			<wfw:commentRss>http://ericsilva.org/2011/02/24/code-review-does-not-have-to-suck/feed/</wfw:commentRss>
		<slash:comments>0</slash:comments>
		</item>
		<item>
		<title>Peer Code Review: An Agile Process</title>
		<link>http://ericsilva.org/2009/12/10/peer-code-review-an-agile-process/</link>
		<comments>http://ericsilva.org/2009/12/10/peer-code-review-an-agile-process/#comments</comments>
		<pubDate>Thu, 10 Dec 2009 11:55:18 +0000</pubDate>
		<dc:creator>Eric Silva</dc:creator>
				<category><![CDATA[Computers]]></category>
		<category><![CDATA[Observations]]></category>
		<category><![CDATA[Programming]]></category>
		<category><![CDATA[Agile]]></category>
		<category><![CDATA[Code Collaborator]]></category>
		<category><![CDATA[LinkedIn]]></category>
		<category><![CDATA[Peer Code Review]]></category>
		<category><![CDATA[Smart Bear]]></category>

		<guid isPermaLink="false">http://ericsilva.org/2009/12/10/peer-code-review-an-agile-process/</guid>
		<description><![CDATA[Smart Bear Software recently released a white paper discussing the misconception that peer code review is a hindrance to Agile development methodologies.&#160; For anyone who regularly performs peer code reviews, would like to start performing them, or thinks they are an obstacle when it comes to Agile development should read this paper. The paper talks [...]]]></description>
			<content:encoded><![CDATA[<p><a href="http://www.smartbear.com" target="_blank">Smart Bear Software</a> recently released a <a href="http://smartbear.com/docs/Peer-Code-Review_An-Agile-Proces.pdf" target="_blank">white paper</a> discussing the misconception that peer code review is a hindrance to Agile development methodologies.&#160; For anyone who regularly performs peer code reviews, would like to start performing them, or thinks they are an obstacle when it comes to Agile development should read this paper.</p>
<p>The paper talks about the history of code review, how code review aligns with Agile, types of lightweight code review, and techniques to perform optimized code reviews.&#160; Some of the key statements that I took away from the paper are this:</p>
<ul>
<li><strong>Code review allows for “continuous attention to technical excellence and good design”.</strong>&#160; These enhances the agility of the code, the developers working on the code, and the overall Agile process.</li>
<li><strong>Code review “promotes sustainable development”.</strong>&#160; The “bus number” concept is one that I use consistently when promoting peer code review in my own workspace and corporate environment.&#160; The white paper explains it simply, “How many team members would have to get struck by a bus before no one was left that understood the code?&#160; If the bus number for a section of the code is less than two, then that’s a problem.”</li>
<li>The final take-away comes from the Agile Manifesto Principles itself, “The best architectures, requirements, and design emerge from self-organizing teams.”&#160; The same is true of quality peer code review; <strong>“If peer code review is mandated by someone outside the team, its chance of success decreases.&#160; If team members do not want code review to succeed, it <em>will</em> fail.”</strong></li>
</ul>
<ul>I use <a href="http://smartbear.com/codecollab.php" target="_blank">Code Collaborator</a>.&#160; I think it’s the best tool out there for performing peer code review, especially with distributed development teams.&#160; I think peer code review allows developers to become better developers through the visibility and social nature of performing the review itself.&#160; My opinions may be a bit biased, based upon my experiences with Smart Bear Software, but, in fairness, this white paper discusses the enormous benefits of peer code review without discussing specific products. It only discusses the principals and observed benefits, and is in no way a sales pitch to buy their product.</ul>
]]></content:encoded>
			<wfw:commentRss>http://ericsilva.org/2009/12/10/peer-code-review-an-agile-process/feed/</wfw:commentRss>
		<slash:comments>0</slash:comments>
		</item>
	</channel>
</rss>

