<?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>Blog - ShiftEleven &#187; codereview</title>
	<atom:link href="http://shifteleven.com/articles/tag/codereview/feed" rel="self" type="application/rss+xml" />
	<link>http://shifteleven.com</link>
	<description>Just another WordPress weblog</description>
	<lastBuildDate>Sat, 19 Sep 2009 11:19:15 +0000</lastBuildDate>
	<language>en</language>
	<sy:updatePeriod>hourly</sy:updatePeriod>
	<sy:updateFrequency>1</sy:updateFrequency>
	<generator>http://wordpress.org/?v=3.0-alpha</generator>
		<item>
		<title>Code Reviews &#8211; Inherit in Git</title>
		<link>http://shifteleven.com/articles/2008/05/17/code-reviews-inherit-in-git</link>
		<comments>http://shifteleven.com/articles/2008/05/17/code-reviews-inherit-in-git#comments</comments>
		<pubDate>Sat, 17 May 2008 23:48:30 +0000</pubDate>
		<dc:creator>K. Adam Christensen</dc:creator>
				<category><![CDATA[Software Development]]></category>
		<category><![CDATA[Version Control]]></category>
		<category><![CDATA[codereview]]></category>
		<category><![CDATA[git]]></category>

		<guid isPermaLink="false">http://shifteleven.com/?p=77</guid>
		<description><![CDATA[Code Reviews are common in software development.  One programmer reviews another&#8217;s code to find potential issues or to see if the developer could have used something that the system already provided.
With multiple programmers, you can probably expect a versioning system, perhaps a centralized versioning system like SVN or Perforce.  With that system, each [...]]]></description>
			<content:encoded><![CDATA[<p><a href="http://en.wikipedia.org/wiki/Code_review">Code Reviews</a> are common in software development.  One programmer reviews another&#8217;s code to find potential issues or to see if the developer could have used something that the system already provided.</p>
<p>With multiple programmers, you can probably expect a versioning system, perhaps a centralized versioning system like SVN or Perforce.  With that system, each programmer would make his/her changes and then would check in his/her changes into the repository.  From there, the programmer&#8217;s peers would review the code.</p>
<p>What&#8217;s the problem with this system?  The code is already checked in.  The pressure is on the peers to ensure that the code is good, and it&#8217;s likely that this could slip through the cracks.  I mean, those peers are working on getting 10 features complete themselves.</p>
<h3>Enter Git</h3>
<p>Well, I would argue that Git has a strong code review process built right in.  This is due to the process of how distributed version control works.</p>
<p><span id="more-77"></span></p>
<p>Programmer <strong>A</strong> creates a <a href="http://railsontherun.com/2008/3/3/how-to-use-github-and-submit-a-patch">fork</a> of the master project. <strong>A</strong> makes his changes and commit his code; however, the code is committed to his local repository and not the master origin.  This is because <strong>A</strong> does not have write access to that repository.</p>
<p><strong>A</strong> has to contact programmer <strong>B</strong>, who has write access. <strong>A</strong> tells <strong>B</strong> that he should pull his code.  Since <strong>B</strong> has to pull the code, this means that <strong>B</strong> has to review the code coming in.</p>
<p>It&#8217;s this workflow that promotes code review.  Now that&#8217;s not to say that you can&#8217;t do code reviews in a centralized model, it just fits a little better in a distributed model</p>
]]></content:encoded>
			<wfw:commentRss>http://shifteleven.com/articles/2008/05/17/code-reviews-inherit-in-git/feed</wfw:commentRss>
		<slash:comments>0</slash:comments>
		</item>
	</channel>
</rss>
