ITK/Gerrit/ReviewPrimer: Difference between revisions
From KitwarePublic
Jump to navigationJump to search
(Created page with "Gerrit code reviews can be relatively straightforward, but also give the reviewer a chance to look deep into proposed changes building and testing those changes locally. This pr...") |
|||
| Line 3: | Line 3: | ||
==Prerequisites== | ==Prerequisites== | ||
This primer assumes a working knowledge of how to build ITK. No details will be given as to how to build and test the code (but they [[ITK_Getting_Started|can be found here]]). This primer also assumes a working knowledge of git. Please see the [[ITK/Git|ITK git pages]] and/or git books ([http://progit.org/book/ ProGit], [http://book.git-scm.com/|git Community Book], or [http://www.amazon.com/Version-Control-Git-Collaborative-Development/dp/0596520123 Version Control withGit]). | This primer assumes a working knowledge of how to build ITK. No details will be given as to how to build and test the code (but they [[ITK_Getting_Started|can be found here]]). This primer also assumes a working knowledge of git. Please see the [[ITK/Git|ITK git pages]] and/or git books ([http://progit.org/book/ ProGit], [http://book.git-scm.com/|git Community Book], or [http://www.amazon.com/Version-Control-Git-Collaborative-Development/dp/0596520123 Version Control withGit]). | ||
Also required is a Gerrit change to review. We will go through a [http://review.source.kitware.com/76 real life code review] of a change submitted to Gerrit. | |||
==Select an open change== | |||
Revision as of 21:34, 20 September 2010
Gerrit code reviews can be relatively straightforward, but also give the reviewer a chance to look deep into proposed changes building and testing those changes locally. This primer is intended to guide the reader through the process of a deep review of a code change.
Prerequisites
This primer assumes a working knowledge of how to build ITK. No details will be given as to how to build and test the code (but they can be found here). This primer also assumes a working knowledge of git. Please see the ITK git pages and/or git books (ProGit, Community Book, or Version Control withGit).
Also required is a Gerrit change to review. We will go through a real life code review of a change submitted to Gerrit.
