mirror of
https://github.com/ceph/ceph
synced 2025-02-22 18:47:18 +00:00
Merge PR #29509 into master
* refs/pull/29509/head: doc: indicate imperative mood for commit titles Reviewed-by: Nathan Cutler <ncutler@suse.com> Reviewed-by: Kefu Chai <kchai@redhat.com> Reviewed-by: Sage Weil <sage@redhat.com>
This commit is contained in:
commit
bbd63e0a7c
@ -397,12 +397,20 @@ could be "doc:", "osd:", or "common:". One can use::
|
||||
|
||||
git log
|
||||
|
||||
for more examples. Please use this "subsystem: short description"
|
||||
convention for naming pull requests (PRs) also, as it feeds directly
|
||||
into the script that generates release notes and it's tedious to clean
|
||||
up at release time. This document places no limit on the length of PR
|
||||
titles, but be aware that they are subject to editing as part of the
|
||||
release process.
|
||||
for more examples. It is also conventional to use the imperative mood in the
|
||||
commit title. For example::
|
||||
|
||||
mds: add perf counter for finisher of MDSRank
|
||||
|
||||
or::
|
||||
|
||||
osd: make the ClassHandler::mutex private
|
||||
|
||||
For GitHub, please use this "subsystem: short description" convention for
|
||||
naming pull requests (PRs). These titles feed directly into the script that
|
||||
generates release notes and it is tedious to clean up non-conformant PR titles
|
||||
at release time. This document places no limit on the length of PR titles, but
|
||||
be aware that they are subject to editing as part of the release process.
|
||||
|
||||
Commit message
|
||||
^^^^^^^^^^^^^^
|
||||
|
Loading…
Reference in New Issue
Block a user