Changes between Initial Version and Version 1 of ticket/1347/review

Timestamp:

2015-05-29T15:01:27+02:00 (9 years ago)

Author:

timgraham

Comment:

--

ticket/1347/review

@@ +1 @@
+Last edited [[Timestamp]]
+
+[[PageOutline]]
+
+== For completion by the Sci/Tech/Code reviewer ==
+'''Reviewer:''' [ Enter your name here ]
+
+=== Ticket Details, Documentation and Code changes ===
+||Do you understand the area of code being altered and the reasoning why it is being altered?||YES/NO||
+||Do the proposed code changes correspond with the stated reason for the change?||YES/NO||
+||Is the in-line documentation accurate and sufficient?||YES/NO||
+||Do the code changes comply with NEMO coding standards?||YES/NO||
+||Is the Ticket documented with sufficient detail for others to understand the impact of the change?||YES/NO||
+||Does any corresponding external documentation require updating?||YES/NO||
+||If yes, which docs and have the updates been drafted?||YES/NO||
+||Are namelist changes required for this change?||YES/NO||
+||If yes, have they been done?||YES/NO||
+||Has a completed Ticket Summary template been appended to the ticket to aid code reviews||YES/NO||
+||Does this summary correspond with your understanding of the full ticket?||YES/NO||
+
+Ticket, Documentation and Code comments
+
+Add specific Ticket, Documentation and code comments here
+
+=== Testing ===
+||Has the NVTK and other jobs been tested with this change?||YES/NO||
+||Have the required bit comparability tests been run?||YES/NO||
+||Can this change be shown to have a null impact? (if option not selected)||YES/NO||
+||If no, is reason for the change valid/understood?||YES/NO/NA||
+||If no, ensure that the ticket details the impact this change will have on model configurations .||YES/NO/NA||
+||Is this change expected to preserve all diagnostics?||YES/NO||
+||If no, is reason for the change valid/understood?||YES/NO/NA||
+||Are there significant changes in run time/memory?||YES/NO||
+
+Testing Comments
+
+Add specific testing comments here
+
+Add specific testing comments here
+
+=== Code Review ===
+||Do the code changes comply with NEMO coding standards?||YES/NO||
+||Are code changes consistent with the design of NEMO?||YES/NO||
+||Is the code free of unwanted TABs?||YES/NO||
+||Has the code been wholly (100%) produced by NEMO developers working on NEMO?||YES/NO||
+||If no, ensure collaboration agreement has been added to the ticket keywords||||
+
+Add specific code comments or suggested alterations here.
+
+=== Review Summary ===
+Add summary here
+
+=== Approval for the trunk ===
+YES/NO
+
+The code reviewer may approve the change for the NEMO trunk when:
+
+ 1. their requests/comments have been addressed satisfactorily.
+ 1. the above check-list has been completed.
+
+or the code reviewer may choose to reject & assign the change back to the code author.