From cdf13e7ca488eb725739ec94371ec3145da0f090 Mon Sep 17 00:00:00 2001 From: Martin Robinson Date: Sun, 12 Apr 2026 10:31:59 +0200 Subject: [PATCH] Make the pull request template more explicit (#44135) There seem to be some common misconceptions about how to use the pull request template such as: - The `Fixes` field going before the pull request description. - The `Testing` field being used to describe how the fix was tested locally. This change attmepts to make the meaning of the fields of the template a little bit more explicit to avoid confusion. Testing: This is just updating the pull request template so no tests necessary. Signed-off-by: Martin Robinson Co-authored-by: Martin Robinson --- PULL_REQUEST_TEMPLATE.md | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/PULL_REQUEST_TEMPLATE.md b/PULL_REQUEST_TEMPLATE.md index 9e531904581..aad8a235519 100644 --- a/PULL_REQUEST_TEMPLATE.md +++ b/PULL_REQUEST_TEMPLATE.md @@ -1,4 +1,5 @@ -*Describe the changes that this pull request makes here. This will be the commit message.* +*Describe the changes that this pull request makes here. This will be the commit message +and it should go at the top of the pull request.* -Testing: *Describe how this pull request is tested or why it doesn't require tests* -Fixes: *Link to an issue this pull requests fixes or remove this line if there is no issue* +Testing: *Describe the new automated tests that cover this change or explain why it doesn't require tests.* +Fixes: *Link to an issue this pull request fixes or remove this line if there is no issue.*