summaryrefslogtreecommitdiff
path: root/Documentation/SubmittingPatches
diff options
context:
space:
mode:
authorJunio C Hamano <gitster@pobox.com>2022-01-27 19:02:57 (GMT)
committerJunio C Hamano <gitster@pobox.com>2022-01-28 01:50:04 (GMT)
commitfa1101afb66dfb3ad325ad135b7bed59e4067dd5 (patch)
treee9153573c60de7a7baab55222ad2a3d97fdcf077 /Documentation/SubmittingPatches
parent89bece5c8c96f0b962cfc89e63f82d603fd60bed (diff)
downloadgit-fa1101afb66dfb3ad325ad135b7bed59e4067dd5.zip
git-fa1101afb66dfb3ad325ad135b7bed59e4067dd5.tar.gz
git-fa1101afb66dfb3ad325ad135b7bed59e4067dd5.tar.bz2
SubmittingPatches: write problem statement in the log in the present tense
We give a guidance for proposed log message to write problem statement first, followed by the reasoning behind, and recipe for, the solution. Clarify that we describe the situation _before_ the proposed patch is applied in the present tense (not in the past tense e.g. "we used to do X, but thanks to this commit we now do Y") for consistency. Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'Documentation/SubmittingPatches')
-rw-r--r--Documentation/SubmittingPatches7
1 files changed, 7 insertions, 0 deletions
diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
index 92b80d9..7225a0f 100644
--- a/Documentation/SubmittingPatches
+++ b/Documentation/SubmittingPatches
@@ -142,6 +142,13 @@ The body should provide a meaningful commit message, which:
. alternate solutions considered but discarded, if any.
+[[present-tense]]
+The problem statement that describes the status quo is written in the
+present tense. Write "The code does X when it is given input Y",
+instead of "The code used to do Y when given input X". You do not
+have to say "Currently"---the status quo in the problem statement is
+about the code _without_ your change, by project convention.
+
[[imperative-mood]]
Describe your changes in imperative mood, e.g. "make xyzzy do frotz"
instead of "[This patch] makes xyzzy do frotz" or "[I] changed xyzzy