ITK/Release 4/Global Code Review/Procedure: Difference between revisions

From KitwarePublic
Jump to navigationJump to search
No edit summary
Line 4: Line 4:
* It contains empty text files for every ITK source code file
* It contains empty text files for every ITK source code file
* As developers review every source file, comments and observations will be registered in the text files of the parallel Git repository.
* As developers review every source file, comments and observations will be registered in the text files of the parallel Git repository.
= Particular Cases =
== Review Directory ==
* The review directory will be separated from the toolkit
* It will be managed as an External module temporarily
* Commonly used classes from Review
** Will be prioritized for code reviews and
** Will be moved to their final destination in the toolkit
An important goal in this case, is to make less common (actually make very rare) that users have to turn the full Review directory on, just to get some classes out of it. We will identify the useful classes, clean them up, fit them up to ITK standards, and move them to a permanent module inside of the toolkit.


== Historical Background ==
== Historical Background ==


* [[ITK_Release_4/Modularization/Code Reviews/Process|Suggested Processes]]
* [[ITK_Release_4/Modularization/Code Reviews/Process|Suggested Processes]]

Revision as of 15:59, 26 April 2011

Procedure

  • A parallel Git repository has been created.
  • It contains empty text files for every ITK source code file
  • As developers review every source file, comments and observations will be registered in the text files of the parallel Git repository.

Particular Cases

Review Directory

  • The review directory will be separated from the toolkit
  • It will be managed as an External module temporarily
  • Commonly used classes from Review
    • Will be prioritized for code reviews and
    • Will be moved to their final destination in the toolkit

An important goal in this case, is to make less common (actually make very rare) that users have to turn the full Review directory on, just to get some classes out of it. We will identify the useful classes, clean them up, fit them up to ITK standards, and move them to a permanent module inside of the toolkit.

Historical Background