Skip to content

Commit 81de5b0

Browse files
authored
Update Git section of the Peer reveiw checklist
1 parent e698b01 commit 81de5b0

File tree

1 file changed

+5
-0
lines changed
  • general/development/process/peer-review

1 file changed

+5
-0
lines changed

general/development/process/peer-review/index.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -227,6 +227,11 @@ Ensure that:
227227
- The commit matches the [Coding style](../../policies/codingstyle/index.md#git-commits)
228228
- The Git history is clean and the work has been rebased to logical commits; and
229229
- The original author of the work provided as a patch has been given credit within the commit (as author of in the commit message if changes were made).
230+
- Git branches have been provided for all the right Moodle branches. The [supported branches list|https://moodledev.io/general/releases#version-support] will help you decide:
231+
- For improvements or new features, this means a branch based on `main` only.
232+
- For normal bug fixes, this the `main` branch and all the 'Current stable' branches (that are affected by the bug).
233+
- For security bugs, this is `main` and all the 'Current stable' and 'Current security' branches (but note the [special process for handling security fixes|https://moodledev.io/general/development/process#security-issues]).
234+
- For improvements or new features in the Behat on PHPunit integration, this should be done on `main` and all the 'Current stable' and 'Current security' branches, providing the backporting is not too difficult.
230235

231236
See also the [Commit cheat sheet](https://docs.moodle.org/dev/Commit_cheat_sheet) for further guidance.
232237

0 commit comments

Comments
 (0)