2654
Comment:
|
← Revision 25 as of 2010-08-16 10:24:56 ⇥
2842
converted to 1.6 markup
|
Deletions are marked like this. | Additions are marked like this. |
Line 6: | Line 6: |
<See [:Code Review Process#CodeReviewPagePerPage:Code Review Page per Class/JSP-page] for description of how to name use this template.> | <See [[Code Review Process#CodeReviewPagePerPage|Code Review Page per Class/JSP-page]] for description of how to name use this template.> |
Line 10: | Line 10: |
||<bgcolor="#dddddd" style="text-align: center;"> '''[:Code Review Process#TablesFilledPageReview:Date]:''' || <date when the review takes place on form YYYY/MM/DD> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[:Code Review Process#TablesFilledPageReview:Version]:''' || <SVN version number> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[:Code Review Process#TablesFilledPageReview:Lines/parts]:''' || <lines/parts reviewed> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[:Code Review Process#TablesFilledPageReview:Bugs/Feature requests]''' || <link to bug(s)/Feature request(s) partly or fully resolved by change> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[:Code Review Process#TablesFilledPageReview:Release test]''' || <excl./incl. TEST case and item and status> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[:Code Review Process#TablesFilledPageReview:Manuals]''' || <Manual , page(s)/chapter(s)> || |
||<bgcolor="#dddddd" style="text-align: center;"> '''[[Code Review Process#TablesFilledPageReview|Date]]:''' || <date when the review takes place on form YYYY/MM/DD> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[[Code Review Process#TablesFilledPageReview|Version]]:''' || <SVN version number> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[[Code Review Process#TablesFilledPageReview|Lines/parts]]:''' || <lines/parts reviewed> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[[Code Review Process#TablesFilledPageReview|Bugs/Feature requests]]''' || <link to bug(s)/Feature request(s) partly or fully resolved by change> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[[Code Review Process#TablesFilledPageReview|Release test]]''' || <excl./incl. TEST case and item and status> || ||<bgcolor="#dddddd" style="text-align: center;"> '''[[Code Review Process#TablesFilledPageReview|Manuals]]''' || <Manual , page(s)/chapter(s)> || |
Line 17: | Line 17: |
[[BR]] ||<bgcolor="#dddddd" style="text-align: center;"> '''[:Code Review Process#TablesFilledPageReview:Line]''' ||<bgcolor="#dddddd" style="text-align: center;"> '''[:Code Review Process#TablesFilledPageReview:Description]''' ||<bgcolor="#dddddd" style="text-align: center;">'''[:Code Review Process#TablesFilledPageReview:Severity]''' ||<bgcolor="#dddddd" style="text-align: center;">'''[:Code Review Process#TablesFilledPageReview:Status]''' || || <Line number, single, serie or interval> || <Description of review comment> || <Must be either [[BR]] '''''Cosmetic''''' (follow-up not necessary), [[BR]] '''''Minor''''' (follow-up would be nice), [[BR]] '''''Major''''' (follow-up must be done) or [[BR]] '''''Showstopper''''' (bug should be reported) > || <Must be either [[BR]] '''''Ok''''' (no further action), [[BR]] '''''Postponed''''' (must be done at a later stage – this must be noted in the "Done" column of the related [:Code_Review_Process#TableFilledReviewsOverview:Code Review Overview Table Filled for each Iteration], [[BR]] '''''Rejected''''' (will not be done at all) or [[BR]] '''''Obsolete''''' (????)> || ||<bgcolor="#dddddd" style="text-align: center;">'''Release Test''' [[BR]] ~--, incl., excl.-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Manuals''' [[BR]] ~-Quick, Deploy, Conf., |
<<BR>> ||<bgcolor="#dddddd" style="text-align: center;"> '''[[Code Review Process#TablesFilledPageReview|Line]]''' ||<bgcolor="#dddddd" style="text-align: center;"> '''[[Code Review Process#TablesFilledPageReview|Description]]''' ||<bgcolor="#dddddd" style="text-align: center;">'''[[Code Review Process#TablesFilledPageReview|Severity]]''' ||<bgcolor="#dddddd" style="text-align: center;">'''[[Code Review Process#TablesFilledPageReview|Status]]'''||<bgcolor="#dddddd" style="text-align: center;">'''[[Code Review Process#TablesFilledPageReview|Release Test]]'''||||<bgcolor="#dddddd" style="text-align: center;">'''[[Code Review Process#TablesFilledPageReview|Manuals]]'''|| || <Line number, single, serie or interval> || <Description of review comment> || <Must be either <<BR>> '''''Cosmetic''''' (follow-up not necessary), <<BR>> '''''Minor''''' (follow-up would be nice), <<BR>> '''''Major''''' (follow-up must be done) or <<BR>> '''''Showstopper''''' (bug should be reported) > || <Must be either <<BR>> '''''Ok''''' (no further action), <<BR>> '''''Postponed''''' (must be done at a later stage – this must be noted in the "Done" column of the related [[Code_Review_Process#TableFilledReviewsOverview|Code Review Overview Table Filled for each Iteration]], <<BR>> '''''Rejected''''' (will not be done at all) or <<BR>> '''''Obsolete''''' (????)> || <Must contain "excl." or <<BR>> incl. TEST case and item and status ) >|| <Can be blank or contain "Manual" names, page(s) and/or chapter(s) >|| |
Reviews of <name of code review class/jsp page>
<See Code Review Page per Class/JSP-page for description of how to name use this template.>
New reviews are added at the top
Date: |
<date when the review takes place on form YYYY/MM/DD> |
<SVN version number> |
|
<lines/parts reviewed> |
|
<link to bug(s)/Feature request(s) partly or fully resolved by change> |
|
<excl./incl. TEST case and item and status> |
|
<Manual , page(s)/chapter(s)> |
<Line number, single, serie or interval> |
<Description of review comment> |
<Must be either |
<Must be either |
<Must contain "excl." or |
<Can be blank or contain "Manual" names, page(s) and/or chapter(s) > |