-
Notifications
You must be signed in to change notification settings - Fork 9
Zero diff changes
Tom Clune edited this page Aug 29, 2019
·
12 revisions
- Build on discover
- baseline from head of develop branch
- evaluation version from your feature branch
- Perform a 1 day run (Q: do we want to specify which options are selected in
gcm_setup?)- baseline
- evaluation
- Verify results are identical
cdo <baseline-exp>/scratch/... <feature-exp>/scratch/...
- Create a PR and check that there are no merge conflicts.
- If there are merge conflicts, fix them on the feature branch and repeat 1.ii, 2.ii, and 3 until there are none.
- If the PR corresponds to an open issue #xxx, please start the PR subject with
Fixes #xxx
- Add the label "0-diff" in the PR
- CODEOWNERs teams will inspect all code changes to ensure that the changes appear sensible and that implementation quality is acceptable. CODEOWNERs may request changes. If nontrivial changes are required, the verifications above will need to be repeated.
- Gatekeeper will verify the zero-diff nature of the change by repeating the steps above.
- Gatekeeper approves the PR
- Click merge button
- Click confirm button
- Click delete branch button
Note: The head of the develop branch may have changed by the time the gatekeepers get to your PR. Usually this will not affect zero-diff changes, but they may ask you to merge the latest develop into your feature branch and repeat the verification steps above.