Differences between revisions 29 and 30
Revision 29 as of 2008-12-19 14:23:57
Size: 5180
Editor: EldZierau
Comment:
Revision 30 as of 2009-03-02 09:55:40
Size: 6252
Editor: EldZierau
Comment:
Deletions are marked like this. Additions are marked like this.
Line 4: Line 4:
Contains: ~-[#CodeReViewTplCrColumns Code Review Column Description]-~, ~-[#CodeReViewTplCrTable Code Review Table]-~, ~-[#CodeReViewTplOrColumns Other Review Column Description]-~, ~-[#CodeReViewTplOrTable Other Review Table]-~ Contains: ~-[#CodeReViewTplCrTable Code Review Table]-~, ~-[#CodeReViewTplOrTable Other Review Table]-~, ~-[#CodeReViewTplCrColumns Code Review Column Description]-~, ~-[#CodeReViewTplOrColumns Other Review Column Description]-~
Line 6: Line 6:
[[Anchor(CodeReViewTplCrColumns)]] '''''Explanation of the columns in code review table:''''' [[Anchor(CodeReViewTplCrTable)]]
'''''Code Reviews'''''[[BR]]
Description of columns can be found in section [#CodeReViewTplCrColumns Explanation of the columns in Code Review table]. [[BR]]
For details on how to do a code review, see [:Process/Code_Review:Code Review Process]. [[BR]]
Code Reviews are done in the review-tool Crucible: [http://kb-prod-udv-001.kb.dk:8060/ Link to Crucible], export is done via [http://kb-prod-udv-001.kb.dk:8060/plugins/servlet/export Crucible export function] [[BR]]
Template for insertion of a new line in edit mode: [[BR]]
`|| [http://kb-prod-udv-001.kb.dk:8060/cru/NS-XX NS-XX] || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || IssuesFromNsXX || INIT || - || - || - ||`
Line 8: Line 14:
||<bgcolor="#dddddd" style="text-align: center;">'''Review Id''' [[BR]] ~-NS-XX-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Task''' [[BR]] ~-Bug/FR XXX-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Author(s)'''[[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Reviewer(s)''' [[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Review date''' [[BR]] ~-YYYY/MM/DD-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Issues found''' [[BR]] ~-IssuesFromNsXX-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Follow-up''' [[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Done''' [[BR]] ~--, OK, OK-wp-~ ||
|| NS-XX || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || IssuesFromNsXX || INIT || - ||

[[Anchor(CodeReViewTplOrTable)]]
'''''Other Reviews'''''[[BR]]
Description of columns can be found in section [#CodeReViewTplOrColumns Explanation of the columns in document review table].[[BR]]
For details on how to other reviews, see [:Process/Document_Review:Document Review Process].[[BR]]
Template for insertion of a new line in edit mode:[[BR]]
`|| [:DocumentReview/DocNameIdReview:DocName] || Version || Lines || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || INIT || - ||`
||<bgcolor="#dddddd" style="text-align: center;">'''Document''' [[BR]] ~-`[:link:docName]`-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Version''' [[BR]] ~-X.Y/date-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Lines/parts''' [[BR]] ~-All/x-y-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Task''' [[BR]] ~-Bug/FR XXX-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Author(s)'''[[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Reviewer(s)''' [[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Review date''' [[BR]] ~-YYYY/MM/DD-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Follow-up''' [[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Done''' [[BR]] ~--, OK, OK-wp-~ ||
|| [:DocumentReview/DocNameIdReview:DocName] || Version || Lines || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || INIT || - ||

[[Anchor(CodeReViewTplCrColumns)]]
'''''Explanation of the columns in Code Review table:'''''
Line 16: Line 36:

[[Anchor(CodeReViewTplCrTable)]]
'''''Code Reviews'''''[[BR]] For details on how to do a code review, see [:Process/Code Review WithTitle:Code Review Process].[[BR]] Code Reviews are done in the review-tool Crucible: [http://kb-prod-udv-001.kb.dk:8060/ Link to Crucible], export is done via [http://kb-prod-udv-001.kb.dk:8060/plugins/servlet/export Crucible export function] [[BR]] Templete for insertion of a new line in edit mode: `|| NS-XX || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || IssuesFromNsXX || INIT || - ||`

||<bgcolor="#dddddd" style="text-align: center;">'''Review Id''' [[BR]] ~-NS-XX-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Task''' [[BR]] ~-Bug/FR XXX-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Author(s)'''[[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Reviewer(s)''' [[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Review date''' [[BR]] ~-YYYY/MM/DD-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Issues found''' [[BR]] ~-IssuesFromNsXX-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Follow-up''' [[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Done''' [[BR]] ~--, OK, OK-wp-~ ||
|| NS-XX || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || IssuesFromNsXX || INIT || - ||
 * "Release Test": Whether the changes are to be included in the Release test. Has values "'''-'''" if new, "'''incl.'''" if it is to be included in the release test and "'''excl.'''" if it cannot be included in the release test (e.g. if not testable).
 * "Manuals": The manuals that has to be/is updated as a concequence of the change. Has values "'''-'''" if new, "'''Quick'''" for [:Quick Start Manual devel: Quick Start Manual], "'''Deploy'''" for [:Deploy Manual devel: Deploy Manual], "'''Conf.'''" for [:Configuration Manual devel: Configuration Manual], "'''Inst..'''" for [:Installation Manual devel: Installation Manual], "'''Dev.'''" for [:Developer Manual devel: Developer Manual], "'''User'''" for [:User Manual devel: User Manual].
Line 34: Line 50:
[[Anchor(CodeReViewTplOrTable)]]
'''''Other Reviews'''''[[BR]]
For details on how to other reviews, see [:Process/Other Review WithTitle:Other Review Process]. [[BR]] Templete for insertion of a new line in edit mode: `|| [:DocumentReview/DocNameIdReview:DocName] || Version || Lines || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || INIT || - ||`

||<bgcolor="#dddddd" style="text-align: center;">'''Document''' [[BR]] ~-`[:link:docName]`-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Version''' [[BR]] ~-X.Y/date-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Lines/parts''' [[BR]] ~-All/x-y-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Task''' [[BR]] ~-Bug/FR XXX-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Author(s)'''[[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Reviewer(s)''' [[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Review date''' [[BR]] ~-YYYY/MM/DD-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Follow-up''' [[BR]] ~-INIT-~ ||<bgcolor="#dddddd" style="text-align: center;">'''Done''' [[BR]] ~--, OK, OK-wp-~ ||
|| [:DocumentReview/DocNameIdReview:DocName] || Version || Lines || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || INIT || - ||

Reviews in Iteration <XX>BR Use this template to start new review pages: [:IterationReviewsOverviewTemplate:Template]

Contains: [#CodeReViewTplCrTable Code Review Table], [#CodeReViewTplOrTable Other Review Table], [#CodeReViewTplCrColumns Code Review Column Description], [#CodeReViewTplOrColumns Other Review Column Description]

Anchor(CodeReViewTplCrTable) Code ReviewsBR Description of columns can be found in section [#CodeReViewTplCrColumns Explanation of the columns in Code Review table]. BR For details on how to do a code review, see [:Process/Code_Review:Code Review Process]. BR Code Reviews are done in the review-tool Crucible: [http://kb-prod-udv-001.kb.dk:8060/ Link to Crucible], export is done via [http://kb-prod-udv-001.kb.dk:8060/plugins/servlet/export Crucible export function] BR Template for insertion of a new line in edit mode: BR || [http://kb-prod-udv-001.kb.dk:8060/cru/NS-XX NS-XX] || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || IssuesFromNsXX || INIT || - || - || - ||

Review Id BR NS-XX

Task BR Bug/FR XXX

Author(s)BR INIT

Reviewer(s) BR INIT

Review date BR YYYY/MM/DD

Issues found BR IssuesFromNsXX

Follow-up BR INIT

Done BR -, OK, OK-wp

NS-XX

Bug/FR XXX

INIT

INIT

YYYY/MM/DD

IssuesFromNsXX

INIT

-

Anchor(CodeReViewTplOrTable) Other ReviewsBR Description of columns can be found in section [#CodeReViewTplOrColumns Explanation of the columns in document review table].BR For details on how to other reviews, see [:Process/Document_Review:Document Review Process].BR Template for insertion of a new line in edit mode:BR || [:DocumentReview/DocNameIdReview:DocName] ||  Version || Lines || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || INIT || - ||

Document BR [:link:docName]

Version BR X.Y/date

Lines/parts BR All/x-y

Task BR Bug/FR XXX

Author(s)BR INIT

Reviewer(s) BR INIT

Review date BR YYYY/MM/DD

Follow-up BR INIT

Done BR -, OK, OK-wp

[:DocumentReview/DocNameIdReview:DocName]

Version

Lines

Bug/FR XXX

INIT

INIT

YYYY/MM/DD

INIT

-

Anchor(CodeReViewTplCrColumns) Explanation of the columns in Code Review table:

  • "Review Id": The Crucible Review Id which the review is created under in Crucible. Normally name is on form NS-XX, i.e. text to be inserted is [http://kb-prod-udv-001.kb.dk:8060/cru/NS-XX NS-XX]

  • "Task": The assignment or tracker issue that the code has been updated for, e.g. Bug 1512.
  • "Author(s)": The person(s) who have made changes or additions to the code. Only Initials are given, e.g. ELZI.
  • "Reviewer(s)": The person(s) who have not been involved in coding these changes, who will participate in the review.
  • "Review date": Date for review.
  • "Issues found": Link to wiki page with issues found under review (and mandays of work). Normally on form IssuesFromNsXX.
  • "Follow-up": The person who will do the follow-up on the review specified under 'Issues found'.
  • "Done": whether the review follow-up has been done. Has value "-" if new, "OK" if all follow-ups are done, "OK-wp" (with postpones) if follow-ups are done with exceptions that have been postponed.

  • "Release Test": Whether the changes are to be included in the Release test. Has values "-" if new, "incl." if it is to be included in the release test and "excl." if it cannot be included in the release test (e.g. if not testable).

  • "Manuals": The manuals that has to be/is updated as a concequence of the change. Has values "-" if new, "Quick" for [:Quick Start Manual devel: Quick Start Manual], "Deploy" for [:Deploy Manual devel: Deploy Manual], "Conf." for [:Configuration Manual devel: Configuration Manual], "Inst.." for [:Installation Manual devel: Installation Manual], "Dev." for [:Developer Manual devel: Developer Manual], "User" for [:User Manual devel: User Manual].

Anchor(CodeReViewTplOrColumns) Explanation of the columns in document review table:

  • "Document": Link to issue review page for document named with identification of the document, e.g. http://netarchive.dk/suite/AssignmentDeploy1 - See example in [Iteration 36].

  • "Version": The SVN, CVS or date for revision of document/script to be reviewed.
  • "Parts/lines": Specifies the parts of the document/script to review (if less that the whole file).
  • "Task": The assignment or tracker issue that the code has been updated for, e.g. Bug 1512.
  • "Author(s)": The person(s) who have made changes or additions to the code. Only Initials are given, e.g. ELZI.
  • "Reviewer(s)": The person(s) who have not been involved in coding these changes, who will participate in the review.
  • "Review date": Date for review.
  • "Follow-up": The person who will do the follow-up on the review specified under 'Issues found'.
  • "Done": whether the review follow-up has been done. Has value "-" if new, "OK" if all follow-ups are done, "OK-wp" (with postpones) if follow-ups are done with exceptions that have been postponed.

IterationReviewsOverviewTemplate (last edited 2010-08-16 10:25:11 by localhost)