1855
Comment:
|
1529
|
Deletions are marked like this. | Additions are marked like this. |
Line 11: | Line 11: |
||<Crucible Review Id linking to Crucible named NS-<id>> ||<Reference to task caused the code changes that are being reviewed as for example a bugs, a feature request or an assignment> ||<Initials of author(s) of changes in code> ||<Initials of reviewer(s) in code review> ||<Date that review was held on form YYYY/MM/DD> ||<Link to wiki page named IssuesFoundInNs<id> which will contain infomraion on issues found in review from Crucible> ||<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)> '' || || bb || ff || ff || gg || mm ||hh ||gg ||hh|| |
||> ||'''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 || |
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" |
' |
vvxcv |
vcxvxc |
vxcvxc |
vxcvcx |
vxcvx |
vxcvx |
vxcvcx |
vxcvx |
bb |
ff |
ff |
gg |
mm |
hh |
gg |
hh |
For details on how to other reviews, see [:Process/Other Review WithTitle:Code Review Process].