Diff for "Iteration33ReviewsOverview"

Differences between revisions 2 and 3

Reviews in iteration 33

Review of code should happen immediately after coding for a task or bug is done, so as little as possible is based on unreviewed code.

If the reviewer for some reason cannot be contacted within a reasonable timeframe, another reviewer should be chosen.

We don't register review time as separate from coding time.

For details on how to do a review, see ["Code Review Process"].

[:Code Review Process#TableFilledReviewsOverview:Class/Page]	[:Code Review Process#TableFilledReviewsOverview:Version]	[:Code Review Process#TableFilledReviewsOverview:Parts/lines]	[:Code Review Process#TableFilledReviewsOverview:Task]	[:Code Review Process#TableFilledReviewsOverview:Author(s)]	[:Code Review Process#TableFilledReviewsOverview:Reviewer(s)]	[:Code Review Process#TableFilledReviewsOverview:Review date]	[:Code Review Process#TableFilledReviewsOverview:Follow-up]	[:Code Review Process#TableFilledReviewsOverview:Done]
[:CommonDistributeArcrepositoryARCLookupReview:dk.netarkivet.common.distribute.arcrepository.ARCLookup]	212	147-150	Bug #1079	LC

-  ⇤ ← Revision 2 as of 2008-01-11 13:46:13 → 
  Size: 1918
  Editor: EldZierau
  Comment:
+   ← Revision 3 as of 2008-01-11 13:47:18 → ⇥
  Size: 1643
  Editor: EldZierau
  Comment:
-Deletions are marked like this.
+Additions are marked like this.
 Line 12:
-||[[CommonDistributeArcrepositoryARCLookupReview][dk.netarkivet.common.distribute.arcrepository.ARCLookup]] ||212||147-150 ||Bug #1079||LC || ||    <Date that review was held on form YYYY/MM/DD> || <Initials of person to do follow-up on code review> || <Must be either [[BR]] '''''Ok''''' (no further action), [[BR]] '''''Ok w/postponed''''' (no further action within this iteration, but includes postpones to be done later)> ||
+||[:CommonDistributeArcrepositoryARCLookupReview:dk.netarkivet.common.distribute.arcrepository.ARCLookup] ||212||147-150 ||Bug #1079||LC || || ||  ||  ||