Changes between Version 1 and Version 2 of ticket/1306/Review
- Timestamp:
- 2014-09-11T17:35:09+02:00 (10 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
ticket/1306/Review
v1 v2 4 4 5 5 == For completion by the Sci/Tech/Code reviewer == 6 '''Reviewer:''' [ Enter your name here ]6 '''Reviewer:''' Italo Epicoco 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||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?||YES|| 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 14 ||Does any corresponding external documentation require updating?||YES/NO|| 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||YES|| 19 ||Does this summary correspond with your understanding of the full ticket?||YES|| 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?||YES|| 27 ||Have the required bit comparability tests been run?||YES|| 28 ||Can this change be shown to have a null impact? (if option not selected)||YES|| 29 ||If no, is reason for the change valid/understood?||NA|| 30 ||If no, ensure that the ticket details the impact this change will have on model configurations .||NA|| 31 ||Is this change expected to preserve all diagnostics?||YES|| 32 ||If no, is reason for the change valid/understood?||NA|| 33 ||Are there significant changes in run time/memory?||NO|| 34 34 35 35 Testing Comments … … 40 40 41 41 === 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||46 ||If no, ensure collaboration agreement has been added to the ticket keywords|| ||42 ||Do the code changes comply with NEMO coding standards?||YES|| 43 ||Are code changes consistent with the design of NEMO?||YES|| 44 ||Is the code free of unwanted TABs?||YES|| 45 ||Has the code been wholly (100%) produced by NEMO developers working on NEMO?||YES|| 46 ||If no, ensure collaboration agreement has been added to the ticket keywords||-|| 47 47 48 48 Add specific code comments or suggested alterations here. … … 52 52 53 53 === Approval for the trunk === 54 YES /NO54 YES 55 55 56 56 The code reviewer may approve the change for the NEMO trunk when: