From Koha Wiki
Bug-enhancement-patch Workflow describes the different steps required to fix and enhance Koha code.
Different people are involved in the process.
- A bug/enhancement is submitted by the 'bug reporter'.
- A patch is submitted by the 'patch writer' and bug status is changed to Needs Sign-off.
- (optional) The release manager ("RM") pushes it as a Quality Assurance ("QA") branch.
- The patch is tested and signed off by the 'patch signer' and bug status is changed to Signed off.
- The patch is checked by the QA team member, and bug status is set to Passed QA
- The patch is tested and signed off by the RM
- If the patch passes, it is pushed to master by the RM and the status is set to Pushed to master.
- If the Release Maintainer ("RMaint") decides the patch can be pushed to the stable version too, they do that and set the status to Pushed to stable
- The bug is marked resolved/fixed by the 'bug closer'.
- The bug is closed when a release is made containing that patch.
- If the 'patch signer' can't test the patch because it does not apply anymore, the status is set to Patch doesn't apply
- If the 'patch signer' can apply the patch but after some tests sees that there is a functional problem with the patch, the status is set to In Discussion or Failed QA, depending on the kind of the problem the tester has detected. The description from the tester must be as detailed as possible to be able to reproduce the failure.
- If the 'QA manager' has some objections during the QA process, the status is set to Failed QA
- If a patch has been pushed to master and a problem is detected after that, there are two options:
- The patch is reverted and a revised patch is requested on the original bug.
- A new bug is filed, linked to the original one, explaining that the fix is not complete, or has a side-effect. Under no circumstances should bugs in new functionality be reported on the original bug.
- Bug reporter: the person who reported the bug. Can also be the patch writer
- Patch writer: the developer who proposed a patch. Can also be the bug reporter
- RM: the Release Manager (elected position)
- QA Manager: the developer (elected member of the QA team) who says the patch is QA compliant
- Patch signer: the user who tested the patch and saw it works
- Bug closer: the person who checked that the bug is now fixed. Can be the reporter, it's preferred that the patch writer is not the closer.
- A bug report must contain information allowing other people to reproduce the bug and test the patch. Usually, there is a section "steps to reproduce: do this, do that, you'll see this buggy behaviour)
- Patch signer should not be patch writer. Preferably, patch writer and patch signer should not be from the same company or institution.
- Patch signer and QA team member are two different persons.
- Bug closer should not be patch writer, unless the patch was written by the original reporter too. Preferably, bug closer should not be patch signer too.
- if your patch is an ENHancement, please add [ENH] just after the patch number in your patch comment 1st line (example: "Bug 6543 [ENH] I add a very nice feature")
- If a patch is not passed QA for a longer time, the developer can leave a note on the bug or send a mail to the mailing list as a gentle nudge to the QA team members.
- If a patch is not passed QA for one month, then it can be QA'd by someone from the same company or institution. The RM can still ask for a second sign-off in this case.