ITK/Release 4/Modularization/Code Reviews/Process

From KitwarePublic
< ITK‎ | Release 4‎ | Modularization‎ | Code Reviews
Revision as of 15:27, 15 February 2011 by Ibanez (talk | contribs) (Created page with "= Overview = * Define a comma separated file per contractor * Containing ** Filename ** Outcome = Possible Outcomes = For each file, a contractor will review the code followin...")
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to navigationJump to search

Overview

  • Define a comma separated file per contractor
  • Containing
    • Filename
    • Outcome

Possible Outcomes

For each file, a contractor will review the code following the Check list (reference here) and specify one of the following outcomes

  • No change required (key: Good)
  • Minor typos (key: Minor)
  • Doxygen documentation needed (need fixing) (key: Dox)
  • File must be removed (broke, obsolete, irreparable) (key: Kill)
  • File needs refactoring (key: BUG#number)(it must have been logged as a mantis bug by the time the CSV file is updated)


Contractors

Refactoring

For classes to be refactored

the following process should be followed

  • MANTIS Bug
  • Dependencies
  • Migration guide
Retrieved from "https://public.kitware.com/Wiki/index.php?title=ITK/Release_4/Modularization/Code_Reviews/Process&oldid=37618"