Changes between Version 4 and Version 5 of Developers/DevelopingCodeChanges
- Timestamp:
- 2010-02-24T15:31:28+01:00 (14 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Developers/DevelopingCodeChanges
v4 v5 1 1 = '''Developing code change''' = 2 3 2 Last edited [[Timestamp]] 4 3 … … 16 15 1. !'''Science/Tech/Code Review!'''. Review of the change by an expert on the scientific or technical area you are changing and for adherence to coding standards. 17 16 18 19 17 Merging back with the trunk is not described here. 20 18 21 === Plan Your Change === 19 === Plan Your Change === 22 20 A discussion at an early stage with the NEMO Systems team/code owners should help make sure your change is viable, desirable and as easy as possible. 23 21 … … 26 24 27 25 * The !''milestone!'' field should be set to the area the change is being targeted for. 28 * The ticket !'' value!'' field should be set to "development branch"26 * The ticket !''type!'' field should be set to "development branch" 29 27 * Associated wiki page should be created by including the markup !wiki:ticket/xxxx (where xxxx is your ticket number) in your ticket and then following the link. 30 * The ticket should be assigned to the code developer at this stage. 28 * The ticket should be assigned to the code developer at this stage. 31 29 32 30 === Create a Branch === … … 46 44 === Test your Change === 47 45 Test your change appropriately and record the results. The testing required will depend on the change and the potential impact on other configurations. When testing, things to consider include 48 * Testing on a variety of processor configurations (1xM, Nx1 and MxN are recommended) 46 47 * Testing on a variety of processor configurations (1xM, Nx1 and MxN are recommended) 49 48 50 49 * Global and Limited-Area Models 51 50 * Coupled Models? 52 51 * If adding new functionality, remember to test that your new code doesn't change results when it is switched off. 53 54 52 55 53 === Document your Change === … … 75 73 76 74 Once the reviewer is satisfied, he/she should complete the approval section of the review template. 77