Diff for "IterationReviewsOverviewTemplate"

Differences between revisions 6 and 7

Reviews in Iteration <XX>

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 time frame, another reviewer should be chosen.

Implementation time including coding, documentation and review is noted under information on issues found.

For details on how to do a code review, see [:Process/Code Review WithTitle:Code Review Process].

Crucible Review Id	Task	Author(s)	Reviewer(s)	Review date	Issues found	Follow-up	Done
>	Expected "=" to follow "Reference"	Expected "=" to follow "Initials"	Expected "=" to follow "Initials"	Expected "=" to follow "Date"	which will contain infomraion on issues found in review from Crucible>	Expected "=" to follow "Initials"	hgf
vvxcv	vcxvxc	vxcvxc	vxcvcx	vxcvx	vxcvx	vxcvcx	vxcvx

NS-XX	ff	ff	gg	mm	hh	gg	hh
NS-XX	ff	ff	gg	mm	hh	gg	hh

For details on how to other reviews, see [:Process/Other Review WithTitle:Code Review Process].

-  ⇤ ← Revision 6 as of 2008-12-19 11:03:53 → 
  Size: 1529
  Editor: EldZierau
  Comment:
+   ← Revision 7 as of 2008-12-19 11:08:53 → ⇥
  Size: 1577
  Editor: EldZierau
  Comment:
-Deletions are marked like this.
+Additions are marked like this.
 Line 10:
-||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Crucible Review Id''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Task''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Author(s)''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Reviewer(s)''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Review date''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Issues found''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Follow-up''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Done''' ||
||> ||'''Expected "=" to follow "Reference"'''||'''Expected "=" to follow "Initials"'''||'''Expected "=" to follow "Initials"'''||'''Expected "=" to follow "Date"'''||which will contain infomraion on issues found in review from Crucible> ||'''Expected "=" to follow "Initials"'''||'''' ||
||vvxcv||vcxvxc||vxcvxc||vxcvcx||vxcvx||vxcvx||vxcvcx||vxcvx||
||bb ||ff ||ff ||gg ||mm ||hh ||gg ||hh ||
+||'''Crucible Review Id''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Task''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Author(s)''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Reviewer(s)''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Review date''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Issues found''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Follow-up''' ||<bgcolor="#dddddd" style="TEXT-ALIGN: center">'''Done''' ||
||> ||'''Expected "=" to follow "Reference"''' ||'''Expected "=" to follow "Initials"''' ||'''Expected "=" to follow "Initials"''' ||'''Expected "=" to follow "Date"''' ||which will contain infomraion on issues found in review from Crucible> ||'''Expected "=" to follow "Initials"''' ||'''hgf''' ||
||vvxcv ||vcxvxc ||vxcvxc ||vxcvcx ||vxcvx ||vxcvx ||vxcvcx ||vxcvx ||
|| || || || || || || || ||
||NS-XX ||ff ||ff ||gg ||mm ||hh ||gg ||hh ||
||NS-XX ||ff ||ff ||gg ||mm ||hh ||gg ||hh ||