]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
SubmittingPatches: add doc for commit title
authorKefu Chai <kchai@redhat.com>
Wed, 20 Jan 2016 08:54:57 +0000 (16:54 +0800)
committerKefu Chai <kchai@redhat.com>
Thu, 21 Jan 2016 05:12:48 +0000 (13:12 +0800)
explain that we need a prefix in the commit title

Signed-off-by: Kefu Chai <kchai@redhat.com>
SubmittingPatches.rst

index 799b99ce14d907d6a6dff69bff19efab19854d61..27f04f78694520df72366e54dd726bca6eb6da31 100644 (file)
@@ -355,6 +355,16 @@ the convenience of the 'pull request' feature.
 
    Describe the technical detail of the change(s) your patch includes.
 
+   The text up to the first empty line in a commit message is the commit
+   title. Ideally it is a single short line less than 50 characters,
+   summarizing the change. It is required to prefix it with the
+   subsystem or module you are changing. For instance, the prefix
+   could be "doc:", "osd:", or "common:". One can use::
+
+     git log
+
+   for more examples.
+
    Be as specific as possible.  The WORST descriptions possible include
    things like "update driver X", "bug fix for driver X", or "this patch
    includes updates for subsystem X.  Please apply."
@@ -387,6 +397,15 @@ the convenience of the 'pull request' feature.
    get more context of this bug, so she/he can hence update the issue on
    the bug tracker accordingly.
 
+   So a typical commit message for revising the document could look like::
+
+     doc: add "--foo" option to bar
+
+     * update the man page for bar with the newly added "--foo" option.
+     * fix a typo
+
+     Fixes: #12345
+     Signed-off-by: Random J Developer <random@developer.example.org>
 
 4. Separate your changes.