Skip to content

Commit 895a8cf

Browse files
committed
Add guidelines section about backporting
From #7723 (comment)
1 parent 98608c4 commit 895a8cf

File tree

1 file changed

+21
-0
lines changed

1 file changed

+21
-0
lines changed

CONTRIBUTING.rst

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -406,6 +406,27 @@ actual latest release). The procedure for this is:
406406
* Delete the PR body, it usually contains a duplicate commit message.
407407

408408

409+
Who does the backporting
410+
~~~~~~~~~~~~~~~~~~~~~~~~
411+
412+
As mentioned above, bugs should first be fixed on ``master`` (except in rare occasions
413+
that a bug only happens in a previous release). So who should do the backport procedure described
414+
above?
415+
416+
1. If the bug was fixed by a core developer, it is the main responsibility of that core developer
417+
to do the backport.
418+
2. However, often the merge is done by another maintainer, in which case it is nice of them to
419+
do the backport procedure if they have the time.
420+
3. For bugs submitted by non-maintainers, it is expected that a core developer will to do
421+
the backport, normally the one that merged the PR on ``master``.
422+
4. If a non-maintainers notices a bug which is fixed on ``master`` but has not been backported
423+
(due to maintainers forgetting to apply the *needs backport* label, or just plain missing it),
424+
they are also welcome to open a PR with the backport. The procedure is simple and really
425+
helps with the maintenance of the project.
426+
427+
All the above are not rules, but merely some guidelines/suggestions on what we should expect
428+
about backports.
429+
409430
Handling stale issues/PRs
410431
-------------------------
411432

0 commit comments

Comments
 (0)