Changes between Version 95 and Version 96 of DeveloperGuidelines/Git


Ignore:
Timestamp:
01/12/14 10:37:58 (8 years ago)
Author:
Pat Tressel
Comment:

--

Legend:

Unmodified
Added
Removed
Modified
  • DeveloperGuidelines/Git

    v95 v96  
    270270=== Code review on !GitHub ===
    271271
     272''Under construction''
     273
    272274Code reviews help assure that code is correct before it gets accepted into trunk by having other people besides the author examine it, install it, test it, and comment on it.  !GitHub provides some very convenient features for conducting code reviews.
    273275
    274276* The code author can push their changes to !GitHub, which makes it easy for the reviewer to pull their changes into their own repository.
    275 * !GitHub allows commenting on specific lines in a file or in a diff between revisions.  This makes it easy to show exactly what code is being commented on.
     277* !GitHub allows commenting on specific lines in a file or in a diff between revisions.  This makes it easy to show exactly what code is being commented on.  It is not necessary to squash commits or make a pull request at this point, but the reviewer will need a way to see your changes as a whole.
     278
     279To prepare for a code review:
     280
     281* Optionally squash your commits.
     282* Push your changes to !GitHub.
     283* If your change contains more than one commit (and you are not doing a pull request yet)
    276284=== Getting your changes accepted into the trunk repository ===
    277285