Changes between Version 1 and Version 2 of ticket/1611/Review
- Timestamp:
- 2015-12-01T11:13:30+01:00 (8 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
ticket/1611/Review
v1 v2 4 4 5 5 == For completion by the Sci/Tech/Code reviewer == 6 '''Reviewer:''' [ Enter your name here]6 '''Reviewer:''' [ Nicolas Jourdain ] 7 7 8 8 === Ticket Details, Documentation and Code changes === 9 ||Do you understand the area of code being altered and the reasoning why it is being altered?||YES /NO||10 ||Do the proposed code changes correspond with the stated reason for the change?||YES /NO||11 ||Is the in-line documentation accurate and sufficient?|| YES/NO||12 ||Do the code changes comply with NEMO coding standards?||YES /NO||13 ||Is the Ticket documented with sufficient detail for others to understand the impact of the change?||YES /NO||14 ||Does any corresponding external documentation require updating?||YES /NO||9 ||Do you understand the area of code being altered and the reasoning why it is being altered?||YES|| 10 ||Do the proposed code changes correspond with the stated reason for the change?||YES|| 11 ||Is the in-line documentation accurate and sufficient?||NO|| 12 ||Do the code changes comply with NEMO coding standards?||YES|| 13 ||Is the Ticket documented with sufficient detail for others to understand the impact of the change?||YES|| 14 ||Does any corresponding external documentation require updating?||YES|| 15 15 ||If yes, which docs and have the updates been drafted?||YES/NO|| 16 ||Are namelist changes required for this change?||YES /NO||17 ||If yes, have they been done?||YES /NO||18 ||Has a completed Ticket Summary template been appended to the ticket to aid code reviews|| YES/NO||19 ||Does this summary correspond with your understanding of the full ticket?|| YES/NO||16 ||Are namelist changes required for this change?||YES|| 17 ||If yes, have they been done?||YES|| 18 ||Has a completed Ticket Summary template been appended to the ticket to aid code reviews|||| 19 ||Does this summary correspond with your understanding of the full ticket?|||| 20 20 21 21 Ticket, Documentation and Code comments … … 24 24 25 25 === Testing === 26 ||Has the NVTK and other jobs been tested with this change?|| YES/NO||27 ||Have the required bit comparability tests been run?|| YES/NO||28 ||Can this change be shown to have a null impact? (if option not selected)|| YES/NO||29 ||If no, is reason for the change valid/understood?|| YES/NO/NA||30 ||If no, ensure that the ticket details the impact this change will have on model configurations .|| YES/NO/NA||31 ||Is this change expected to preserve all diagnostics?||YES /NO||32 ||If no, is reason for the change valid/understood?|| YES/NO/NA||33 ||Are there significant changes in run time/memory?|| YES/NO||26 ||Has the NVTK and other jobs been tested with this change?||NA|| 27 ||Have the required bit comparability tests been run?||NA|| 28 ||Can this change be shown to have a null impact? (if option not selected)|||| 29 ||If no, is reason for the change valid/understood?|||| 30 ||If no, ensure that the ticket details the impact this change will have on model configurations .|||| 31 ||Is this change expected to preserve all diagnostics?||YES|| 32 ||If no, is reason for the change valid/understood?|||| 33 ||Are there significant changes in run time/memory?||NA|| 34 34 35 35 Testing Comments 36 36 37 Add specific testing comments here 38 39 Add specific testing comments here 37 I did not undertake any test on this branch 40 38 41 39 === Code Review === 42 ||Do the code changes comply with NEMO coding standards?||YES /NO||43 ||Are code changes consistent with the design of NEMO?||YES /NO||44 ||Is the code free of unwanted TABs?||YES /NO||45 ||Has the code been wholly (100%) produced by NEMO developers working on NEMO?||YES /NO||40 ||Do the code changes comply with NEMO coding standards?||YES|| 41 ||Are code changes consistent with the design of NEMO?||YES|| 42 ||Is the code free of unwanted TABs?||YES|| 43 ||Has the code been wholly (100%) produced by NEMO developers working on NEMO?||YES|| 46 44 ||If no, ensure collaboration agreement has been added to the ticket keywords|||| 47 45 … … 49 47 50 48 === Review Summary === 51 Add summary here49 Good job Pierre 52 50 53 51 === Approval for the trunk === 54 YES /NO52 YES 55 53 56 54 The code reviewer may approve the change for the NEMO trunk when: