[PATCH] docs: Update best practices wrt "Fixes:" and GitLab

Michal Privoznik mprivozn at redhat.com
Mon Oct 24 11:52:26 UTC 2022


We document that a commit fixing an issue tracked in GitLab
should put just "Fixes: #NNN" into its commit message. But when
viewing git log, having full URL which is directly clickable is
more developer friendly and GitLab is capable of handling both.
Therefore, document that users should put full URL, just like
when fixing a bug tracked in other sites.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 docs/best-practices.rst | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/docs/best-practices.rst b/docs/best-practices.rst
index ba8478ab05..a4651c19d0 100644
--- a/docs/best-practices.rst
+++ b/docs/best-practices.rst
@@ -15,11 +15,11 @@ with minimal back-and-forth.
    by any longer description of why your patch makes sense. If the
    patch fixes a regression, and you know what commit introduced
    the problem, mentioning that is useful. If the patch resolves a
-   upstream bug reported in GitLab, put "Fixes: #NNN" in the commit
-   message. For a downstream bug, mention the URL of the bug instead.
-   In both cases also summarize the issue rather than making all
-   readers follow the link. You can use 'git shortlog -30' to get
-   an idea of typical summary lines.
+   upstream bug reported in GitLab, or downstream bug, put
+   "Resolves: $fullURL" of the bug. In both cases also summarize
+   the issue rather than making all readers follow the link. You
+   can use 'git shortlog -30' to get an idea of typical summary
+   lines.
 
 -  Split large changes into a series of smaller patches,
    self-contained if possible, with an explanation of each patch
-- 
2.37.4



More information about the libvir-list mailing list