Diff for "CodeReviewCodePageTemplate"

Differences between revisions 1 and 24 (spanning 23 versions)

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

New reviews are added at the top

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

[:Code Review Process#TablesFilledPageReview:Line]	[:Code Review Process#TablesFilledPageReview:Description]	[:Code Review Process#TablesFilledPageReview:Severity]	[:Code Review Process#TablesFilledPageReview:Status]	[:Code Review Process#TablesFilledPageReview:Release Test]	[: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) >

-  ⇤ ← Revision 1 as of 2008-01-11 13:05:14 → 
  Size: 1851
  Editor: EldZierau
  Comment:
+   ← Revision 24 as of 2009-06-08 12:07:52 → ⇥
  Size: 2828
  Editor: TueLarsen
  Comment:
-Deletions are marked like this.
+Additions are marked like this.
 Line 1:
-#acl NetarkivetGroup:read,write,delete,revert,admin All:
+## page was renamed from Code Review Code Page Template
## page was renamed from Template Code Review Code Page

~+'''Reviews of <name of code review class/jsp page>'''+~

<See [:Code Review Process#CodeReviewPagePerPage:Code Review Page per Class/JSP-page] for description of how to name use this template.>

''New reviews are added at the top''
-Line 7:
+Line 14:
+||<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 9:
+Line 18:
-||<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;"> '''[: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) >||