Diff for "CodeReviewCodePageTemplate"

Differences between revisions 24 and 25

Reviews of <name of code review class/jsp page>

New reviews are added at the top

Date:	<date when the review takes place on form YYYY/MM/DD>
Version:	<SVN version number>
Lines/parts:	<lines/parts reviewed>
Bugs/Feature requests	<link to bug(s)/Feature request(s) partly or fully resolved by change>
Release test	<excl./incl. TEST case and item and status>
Manuals	<Manual , page(s)/chapter(s)>

Line	Description	Severity	Status	Release Test	Manuals
<Line number, single, serie or interval>	<Description of review comment>	<Must be either Cosmetic (follow-up not necessary), Minor (follow-up would be nice), Major (follow-up must be done) or Showstopper (bug should be reported) >	<Must be either Ok (no further action), Postponed (must be done at a later stage – this must be noted in the "Done" column of the related Code Review Overview Table Filled for each Iteration, Rejected (will not be done at all) or Obsolete (????)>	<Must contain "excl." or incl. TEST case and item and status ) >	<Can be blank or contain "Manual" names, page(s) and/or chapter(s) >

-  ⇤ ← Revision 24 as of 2009-06-08 12:07:52 → 
  Size: 2828
  Editor: TueLarsen
  Comment:
+   ← Revision 25 as of 2010-08-16 10:24:56 → ⇥
  Size: 2842
  Editor: localhost
  Comment: converted to 1.6 markup
-Deletions are marked like this.
+Additions are marked like this.
 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:
-||<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:
-[[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) >||
+<<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) >||