Mercurial > repos > public > sbplib_julia
comparison WORKFLOW.md @ 1165:32b4d324fd98 worklowdocs
Add some text about the branch model
| author | Jonatan Werpers <jonatan@werpers.com> |
|---|---|
| date | Wed, 07 Dec 2022 21:42:18 +0100 |
| parents | 74c09b5f63eb |
| children | 6970bffad843 |
comparison
equal
deleted
inserted
replaced
| 1161:74c09b5f63eb | 1165:32b4d324fd98 |
|---|---|
| 1 # Branch model | 1 # Branch model |
| 2 | |
| 3 The `default` branch contains stable code and the goal is that all tests should be passing on this branch. Version of the code are marked with mercurial tags. Changes in the code are developed in named branches which are closed and merged into `default` when the code is ready. During development merging `default` into the development branch is encouraged to avoid complicated conflicts. | |
| 4 | |
| 5 Branches are named using slash separated keywords. The first keyword describes the type of change being pursued in the branch. Important type keywords are | |
| 6 * feature | |
| 7 * bugfix | |
| 8 * refactor | |
| 9 Further keywords may describe where, e.g. what sub package, the change happens. The last keyword should describe the change. | |
| 10 | |
| 11 ## Merging a branch into `default` | |
| 12 The changes in a branch has been reviewed and deemed ready to merge the branch is closed and then merged. | |
| 13 | |
| 14 Before merging a development branch, `default` should be merge into the development branch to make sure the whole state of the code is reviewed and tested before it ends up on `default`. | |
| 15 | |
| 16 With the development branch active the following commands can be used to complete the merging of a development branch. | |
| 17 ```shell | |
| 18 hg merge default | |
| 19 hg commit --close-branch -m "Close before merge" | |
| 20 hg update default | |
| 21 hg merge development/branch | |
| 22 hg commit -m "Merge development/branch" | |
| 23 ``` | |
| 2 | 24 |
| 3 # Review | 25 # Review |
| 4 | 26 |
| 5 ## Checklist for review | 27 ## Checklist for review |
| 6 * Push and pull new changes | 28 * Push and pull new changes |
| 7 * Search and check TODOs | 29 * Search and check TODOs |
| 8 * Search and check TBDs | 30 * Search and check TBDs |
| 9 * Search and check REVIEWs | 31 * Search and check REVIEWs |
| 10 * Review code | 32 * Review code |
| 11 * Review tests | 33 * Review tests |
| 12 * Review Docstrings | 34 * Review docstrings |
| 13 * Render Documenter and check docstrings in browser | 35 * Render Documenter and check docstrings in browser |
| 14 * Run full tests | 36 * Run full tests |